Github user hyunsik commented on the pull request:
https://github.com/apache/tajo/pull/593#issuecomment-122240606
Thank you for your work.
I leave additional trivial comments.
* LogicalPlanRewriteRule in LogicalOptimizer is not used.
* handleRemainingFiltersIfNecessary needs a brief comment to explain its
purpose.
* JoinOrderingUtil includes several unused imports.
* The multi line comments In 63 line in GreedyHeuristicAlgorithm.java
should use ``//`` instead of ``/*``.
* GreedyHeuristicAlgorithm::prepareGraphUpdate needs a brief comment to
its purpose.
The patch seems to be ready to be committed. After your answer, I'll finish
the review.
---
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 feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
