You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@quickstep.apache.org by zuyu <gi...@git.apache.org> on 2018/04/22 04:56:52 UTC

[GitHub] incubator-quickstep pull request #342: DO NOT MERGE: Quickstep-119: Added th...

GitHub user zuyu opened a pull request:

    https://github.com/apache/incubator-quickstep/pull/342

    DO NOT MERGE: Quickstep-119: Added the rule that eliminates a HashJoin to a Selection if possible.

    This PR adds an optimization rule that eliminates a join if at least one side is empty.
    
    TODO: add unit tests.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/zuyu/incubator-quickstep simplify-join-rule

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-quickstep/pull/342.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #342
    
----
commit c670a5bec9374f9c613461114c234eeea25f2de1
Author: Zuyu Zhang <zu...@...>
Date:   2018-04-18T22:15:34Z

    Added the rule that eliminates a HashJoin to a Selection if possible.

----


---

[GitHub] incubator-quickstep pull request #342: Quickstep-119: Added the rule that el...

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

    https://github.com/apache/incubator-quickstep/pull/342


---

[GitHub] incubator-quickstep issue #342: Quickstep-119: Added the rule that eliminate...

Posted by jianqiao <gi...@git.apache.org>.

Github user jianqiao commented on the issue:

    https://github.com/apache/incubator-quickstep/pull/342
  
    LGTM! Merging.


---