velox: [Native] NestedloopJoin with join criteria produces incorrect number of rows.
Bug description
Produces 1 addtional row more than the expected. Issue fails often on cluster with multiple workers.
java.lang.AssertionError: For query:
SELECT * FROM (SELECT * FROM nation WHERE nation.nationkey%2=0)FULL JOIN (SELECT * FROM region WHERE regionkey%2=0) region ON region.regionkey = 2
not equal
Actual rows (up to 100 of 1 extra rows shown, 16 rows in total):
[null, null, null, null, 2, ASIA, ges. thinly even pinto beans ca]
Expected rows (up to 100 of 0 missing rows shown, 15 rows in total):
System information
NA
Relevant logs
No response
About this issue
- Original URL
- State: closed
- Created 10 months ago
- Comments: 18 (5 by maintainers)
Commits related to this issue
- Fix NestedLoopJoinProbe operator finishing before handing mismatch data to the last prober. (#6326) Summary: Fixes https://github.com/facebookincubator/velox/issues/6223. Currently non-last probe op... — committed to codyschierbeck/velox by usurai 10 months ago
- Fix NestedLoopJoinProbe operator finishing before handing mismatch data to the last prober. (#6326) Summary: Fixes https://github.com/facebookincubator/velox/issues/6223. Currently non-last probe op... — committed to codyschierbeck/velox by usurai 10 months ago
- Fix NestedLoopJoinProbe operator finishing before handing mismatch data to the last prober. (#6326) Summary: Fixes https://github.com/facebookincubator/velox/issues/6223. Currently non-last probe op... — committed to codyschierbeck/velox by usurai 10 months ago
- Fix NestedLoopJoinProbe operator finishing before handing mismatch data to the last prober. (#6326) Summary: Fixes https://github.com/facebookincubator/velox/issues/6223. Currently non-last probe op... — committed to ericyuliu/velox by usurai 10 months ago
- Fix NestedLoopJoinProbe operator finishing before handing mismatch data to the last prober. (#6326) Summary: Fixes https://github.com/facebookincubator/velox/issues/6223. Currently non-last probe op... — committed to mikelui/velox by usurai 10 months ago
https://app.circleci.com/pipelines/github/prestodb/presto/6974/workflows/ccf507fc-9314-4833-8211-0ecab8353b90/jobs/22350 If no IntelliJ, you can run e2e by starting a docker container with image
prestocpp/prestocpp-avx-centos:root-20230613and follow steps here in this file.circleci/continue_config.yml.