-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18305/
-----------------------------------------------------------
(Updated Feb. 20, 2014, 6:45 p.m.)
Review request for Tajo.
Bugs: TAJO-614
https://issues.apache.org/jira/browse/TAJO-614
Repository: tajo
Description (updated)
-------
Currently, many parts use LogicalNode::toString() for explaning plans. But, we
already have ExplainLogicalPlanVisitor class to generate pretty print strings.
This patch improves logical planning related parts to use
ExplainLogicalPlanVisitor instead of toString(). For this, I added
PlannerUtil::buildExplainString for generating pretty print explain strings and
simplified obsolete toString() methods of all LogicalNodes. It much improves
the readability of explain string. I expect that it would be helpful for
debugging and users' understanding of query plans.
Diffs
-----
Diff: https://reviews.apache.org/r/18305/diff/
Testing
-------
mvn clean install -Phcatalog-0.12.0
Thanks,
Hyunsik Choi
