[GitHub] incubator-quickstep issue #354: Fixed the union-all elimiation case where so...

2018-05-09 Thread jianqiao
Github user jianqiao commented on the issue: https://github.com/apache/incubator-quickstep/pull/354 LGTM! Merging. ---

[GitHub] incubator-quickstep issue #353: Minor bug fixes and refactors.

2018-05-09 Thread jianqiao
Github user jianqiao commented on the issue: https://github.com/apache/incubator-quickstep/pull/353 LGTM! Merging. ---

[GitHub] incubator-quickstep pull request #347: QUICKSTEP-121: Added the self-join su...

2018-05-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-quickstep/pull/347 ---

[GitHub] incubator-quickstep pull request #353: Minor bug fixes and refactors.

2018-05-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-quickstep/pull/353 ---

[GitHub] incubator-quickstep issue #347: QUICKSTEP-121: Added the self-join support.

2018-05-09 Thread jianqiao
Github user jianqiao commented on the issue: https://github.com/apache/incubator-quickstep/pull/347 LGTM! Merging. Note that the `concretize` signature in `Physical` plans looks somehow cumbersome, we may add a `SubstitutionContext` class to wrap these in a future PR.