Implement mixed equality/conditional semi/anti joins #10037
Conversation
vyasr
added
feature request
|
To reviewers (@karthikeyann and @davidwendt) I would recommend waiting to review until #9917 is merged. I've opened this to allow the Spark team to start testing but this PR currently shows twice the changeset it needs. |
Codecov Report
@@ Coverage Diff @@
## branch-22.04 #10037 +/- ##
================================================
+ Coverage 10.37% 10.42% +0.04%
================================================
Files 119 119
Lines 20149 20607 +458
================================================
+ Hits 2091 2148 +57
- Misses 18058 18459 +401
Continue to review full report at Codecov.
|
…and get first working prototype of semi join.
…uality and conditional tables.
vyasr
force-pushed
the
feature/mixed_semi_anti_joins
branch
from
58c16f2 to
5f0378e
Compare
Yeah we can look into that. We'll have to make a call on what sort of performance hit is acceptable for the compile-time improvement, I can look at previous nullate PRs to assess how we've decided that in the past. |
|
rerun tests |
vyasr
requested review from
rgsl888prabhu and
brandon-b-miller
and removed request for
a team
vyasr
added
3 - Ready for Review
|
@gpucibot merge |
This depends on #9941 and #10037. Adds Java bindings for mixed left semi join and mixed left anti join. Authors: - Jason Lowe (https://github.com/jlowe) Approvers: - Alessandro Bellina (https://github.com/abellina) - Thomas Graves (https://github.com/tgravescs) - Jim Brennan (https://github.com/jbrennan333) URL: #10040
This PR is a follow-up to #9917 and should be merged after that PR. This resolves #9695 and resolves #5401. The implementation here is only a first pass, but in the interest of prioritizing a working feature for the upcoming release I'm postponing making various additional changes (including some breaking ones).