Skip to content
This repository was archived by the owner on May 12, 2021. It is now read-only.

TAJO-1275: Optimizer pushs down non-equi filter as theta join qualifier#327

Closed
sirpkt wants to merge 2 commits into
apache:masterfrom
sirpkt:TAJO-1275
Closed

TAJO-1275: Optimizer pushs down non-equi filter as theta join qualifier#327
sirpkt wants to merge 2 commits into
apache:masterfrom
sirpkt:TAJO-1275

Conversation

@sirpkt
Copy link
Copy Markdown
Contributor

@sirpkt sirpkt commented Jan 2, 2015

This patch prevents non-equi filter condition becomes join qualifier of child nodes.
It can avoid theta join is created as a result of filter push down.

in FilterPushDownRule.java, visitJoin() additionally checks all the join qualifier candidates from pushing down filters whether it is non-equi join qualifier or not.

@sirpkt sirpkt changed the title Tajo-1275: Optimizer pushs down non-equi filter as theta join qualifier TAJO-1275: Optimizer pushs down non-equi filter as theta join qualifier Jan 2, 2015
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be better if you note that this line must be restored when https://issues.apache.org/jira/browse/TAJO-742 is resolved.
The TODO annotation will be helpful for the future restoration.

@jihoonson
Copy link
Copy Markdown
Contributor

Hi @sirpkt
This patch looks good to me.
+1 and thanks for your contribution.

@asfgit asfgit closed this in 9e329a5 Jan 3, 2015
@sirpkt
Copy link
Copy Markdown
Contributor Author

sirpkt commented Jan 3, 2015

Thank you for the review, @jihoonson.
I added the comments about TODO and just committed the patch.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants