Skip to content

Fix semi-join / pipeline breaker on BYTES and BIG_DECIMAL#15531

Merged
yashmayya merged 1 commit intoapache:masterfrom
yashmayya:semi-join-pipeline-breaker-type-fix
Apr 14, 2025
Merged

Fix semi-join / pipeline breaker on BYTES and BIG_DECIMAL#15531
yashmayya merged 1 commit intoapache:masterfrom
yashmayya:semi-join-pipeline-breaker-type-fix

Conversation

@yashmayya
Copy link
Contributor

@yashmayya yashmayya added bugfix multi-stage Related to the multi-stage query engine labels Apr 14, 2025
@codecov-commenter
Copy link

codecov-commenter commented Apr 14, 2025

Codecov Report

Attention: Patch coverage is 93.33333% with 1 line in your changes missing coverage. Please review.

Project coverage is 62.94%. Comparing base (fb02f04) to head (7375b57).
Report is 3 commits behind head on master.

Files with missing lines Patch % Lines
...ry/runtime/plan/server/ServerPlanRequestUtils.java 93.33% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##             master   #15531   +/-   ##
=========================================
  Coverage     62.93%   62.94%           
  Complexity     1382     1382           
=========================================
  Files          2849     2852    +3     
  Lines        161679   161710   +31     
  Branches      24766    24733   -33     
=========================================
+ Hits         101752   101783   +31     
- Misses        52251    52280   +29     
+ Partials       7676     7647   -29     
Flag Coverage Δ
custom-integration1 100.00% <ø> (ø)
integration 100.00% <ø> (ø)
integration1 100.00% <ø> (ø)
integration2 0.00% <ø> (ø)
java-11 62.91% <93.33%> (+0.01%) ⬆️
java-21 62.93% <93.33%> (+0.02%) ⬆️
skip-bytebuffers-false 62.93% <93.33%> (+0.03%) ⬆️
skip-bytebuffers-true 62.91% <93.33%> (+0.01%) ⬆️
temurin 62.94% <93.33%> (+<0.01%) ⬆️
unittests 62.93% <93.33%> (+<0.01%) ⬆️
unittests1 55.94% <93.33%> (+0.03%) ⬆️
unittests2 33.64% <0.00%> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@yashmayya yashmayya merged commit 1515f49 into apache:master Apr 14, 2025
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix multi-stage Related to the multi-stage query engine

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Semi join pipeline breaker fails with types like BYTES and BIG_DECIMAL in the multi-stage engine

3 participants