GitHub user zuyu opened a pull request:
https://github.com/apache/incubator-quickstep/pull/81
Refactored QueryProcessor.
Assigned to @jianqiao.
This PR reverted `optimizer` per query, and avoids memory leaks if
encountered any exceptions.
You can merge this pull request in
Github user jianqiao commented on the issue:
https://github.com/apache/incubator-quickstep/pull/79
That sounds good. Note that the refactoring needs quite some work to pass
`optimizer_context` around, especially in `ExecutionGenerator`.
---
If your project is set up for it, you can r
Github user zuyu commented on the issue:
https://github.com/apache/incubator-quickstep/pull/79
@jianqiao Thank you for your comments.
What do you think if we move `OptimizerContext` from `Optimizer` to
`QueryHandle`, which is unique per query?
---
If your project is set up f
GitHub user zuyu opened a pull request:
https://github.com/apache/incubator-quickstep/pull/80
Added PolicyEnforcer implementation for the distributed version.
The functionality was tested by #70.
You can merge this pull request into a Git repository by running:
$ git pull https
Github user zuyu commented on the issue:
https://github.com/apache/incubator-quickstep/pull/76
@jianqiao Please submit another PR to fix `iwyu` issues for all the files
touched by this PR, including adding `` in
`query_execution/QueryExecutionTypedefs.hpp`, and removing `` in both
`
Github user jianqiao commented on the issue:
https://github.com/apache/incubator-quickstep/pull/79
@zuyu The `Generator`'s can be stateless (need some refactoring), but each
query should have its own `OptimizerContext`.
---
If your project is set up for it, you can reply to this emai
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-quickstep/pull/78
---
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 feature
enabled and wishes so, or if the fea
Github user pateljm commented on the issue:
https://github.com/apache/incubator-quickstep/pull/77
LGTM. Merging.
---
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 feature
enabled and wishe
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-quickstep/pull/77
---
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 feature
enabled and wishes so, or if the fea