GitHub user dylanpbacon opened a pull request:
https://github.com/apache/incubator-quickstep/pull/300
Hash-Join-Fuse: Feature added and tests modified.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/dylanpbacon/incubator
Github user dylanpbacon commented on a diff in the pull request:
https://github.com/apache/incubator-quickstep/pull/300#discussion_r140610972
--- Diff: query_optimizer/strategy/Join.cpp ---
@@ -371,6 +373,7 @@ void Join::addHashJoin(const logical::ProjectPtr
&logical_pro
Github user dylanpbacon commented on the issue:
https://github.com/apache/incubator-quickstep/pull/300
I believe all bugs have been fixed except the LIPFilter applying to
modified HashJoins. Jianqiao will be taking care of that I think. Below are the
updated benchmark values
GitHub user dylanpbacon opened a pull request:
https://github.com/apache/incubator-quickstep/pull/322
Generalized hash
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/dylanpbacon/incubator-quickstep
Generalized-Hash
Github user dylanpbacon commented on the issue:
https://github.com/apache/incubator-quickstep/pull/322
Not meant for immediate merging, just collaboration and seeing what's
changed. If there's a better way to do this let me know and this PR will be
closed.
---
Github user dylanpbacon commented on the issue:
https://github.com/apache/incubator-quickstep/pull/300
Rebased this with master, this branch is now up to date. LIP Filters still
need to be fixed to work properly with this feature.
---
Github user dylanpbacon commented on the issue:
https://github.com/apache/incubator-quickstep/pull/300
This PR is now consistent with the changes @jianqiao made. It is ready to
be merged, I believe. I do not have rights for this yet I think.
---
Github user dylanpbacon closed the pull request at:
https://github.com/apache/incubator-quickstep/pull/322
---
