Github user jianqiao commented on the issue:
https://github.com/apache/incubator-quickstep/pull/354
LGTM! Merging.
---
Github user jianqiao commented on the issue:
https://github.com/apache/incubator-quickstep/pull/353
LGTM! Merging.
---
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-quickstep/pull/347
---
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-quickstep/pull/353
---
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.