[WIP] Add support for BNLJ #343
Conversation
|
Thanks for the contribution @singhpk234. I see this is marked WIP so will hold off on reviewing until it is ready for review. Let us know if you need any help or have questions. |
|
I saw this is in WIP for a while. @singhpk234 Would you like to continue working on it? Or we can have someone take it over? Thanks. |
|
Apologies @andygrove @viirya got swamped by internal company work and lost track of this pr, i would able to make it ready for review by EOW for sure (thinking of utilizing this weekend for the same) if it's fine ? |
|
No worries. Just want to make sure if you still work on it. If not, we could have someone pick it up. Thanks. |
|
Thanks @viirya for sure I am on it from now ! |
singhpk234
force-pushed
the
feature/support-bnlj
branch
from
5e9e8c5
to
daa2f6b
Compare
|
Moving this to draft for now since it has been inactive for a while |
|
@singhpk234 I will close this PR for now since it has been inactive for a while. Feel free to reopen this or create a new PR if you continue work on this. |
Which issue does this PR close?
Closes #.#198
Rationale for this change
What changes are included in this PR?
Support for BNLJ, this supports is missing in supported joins
How are these changes tested?
UT