quickstep-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From zuyu <...@git.apache.org>
Subject [GitHub] incubator-quickstep pull request #347: DO NOT MERGE: QUICKSTEP-121: Added th...
Date Mon, 30 Apr 2018 06:34:31 GMT
GitHub user zuyu opened a pull request:

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

    DO NOT MERGE: QUICKSTEP-121: Added the self-join support.

    This PR added the self-join support by assigning the join side info, instead of relying
on the relation id.
    
    Note that now `NestedLoopsJoin` has to use the same term as `HashJoin` w.r.t. join side.
In the other words, the left side is the `probe`, while the right is the `build`. Also, the
join result pair is in the format of `<build-tid, probe-tid>`.

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

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

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

    https://github.com/apache/incubator-quickstep/pull/347.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 #347
    
----
commit 4055d65a207b63775f2cf4c4841b33da15d70cd2
Author: Zuyu Zhang <zuyu@...>
Date:   2018-04-30T03:50:02Z

    Added the self-join support.

----


---

Mime
View raw message