Impala Public Jenkins has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/11980 )

Change subject: IMPALA-7881: Visualize AST for easier debugging
......................................................................

IMPALA-7881: Visualize AST for easier debugging

Provides a debug-time class to visualize the analysis-time AST for use
by developers. Uses a very simple JSON-like output. See IMPALA-7881 for
an example. Emphasize simplicity over elegance, many refinements can
follow.

Tests: This is test only code that is not even called from anywhere. A
developer must insert a temporary call when doing debugging. As a
result, the code cannot impact any other part of the code base.

Change-Id: Ia083c63953ceec0abedd2e0753b9444c19fcd114
Reviewed-on: http://gerrit.cloudera.org:8080/11980
Reviewed-by: Impala Public Jenkins <[email protected]>
Reviewed-by: Lars Volker <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>
---
A fe/src/test/java/org/apache/impala/util/treevis/AstPrinter.java
A fe/src/test/java/org/apache/impala/util/treevis/TreePrinter.java
A fe/src/test/java/org/apache/impala/util/treevis/Visualizer.java
3 files changed, 393 insertions(+), 0 deletions(-)

Approvals:
  Impala Public Jenkins: Looks good to me, approved; Verified
  Lars Volker: Looks good to me, but someone else must approve

--
To view, visit http://gerrit.cloudera.org:8080/11980
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia083c63953ceec0abedd2e0753b9444c19fcd114
Gerrit-Change-Number: 11980
Gerrit-PatchSet: 3
Gerrit-Owner: Paul Rogers <[email protected]>
Gerrit-Reviewer: Bharath Vissapragada <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Lars Volker <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>

Reply via email to