[GitHub] incubator-quickstep issue #257: Added Partition Rule for HashJoin.

2017-06-17 Thread zuyu
Github user zuyu commented on the issue: https://github.com/apache/incubator-quickstep/pull/257 Merging after discussing with @jianqiao. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] incubator-quickstep issue #257: Added Partition Rule for HashJoin.

2017-06-15 Thread jianqiao
Github user jianqiao commented on the issue: https://github.com/apache/incubator-quickstep/pull/257 Good refactoring. LGTM! Except one place in `PhysicalGenerator.cpp`. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If