Till Westmann has uploaded this change for review. ( 
https://asterix-gerrit.ics.uci.edu/3384


Change subject: MB-34087: show aggregate functions in JSON plan
......................................................................

MB-34087: show aggregate functions in JSON plan

Change-Id: Ie4ba74d8455844e3913a4d4b5e2761b5ed23028b
---
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/prettyprint/LogicalOperatorPrettyPrintVisitorJson.java
1 file changed, 4 insertions(+), 0 deletions(-)



  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/84/3384/1

diff --git 
a/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/prettyprint/LogicalOperatorPrettyPrintVisitorJson.java
 
b/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/prettyprint/LogicalOperatorPrettyPrintVisitorJson.java
index e56a8bd..f4e8003 100644
--- 
a/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/prettyprint/LogicalOperatorPrettyPrintVisitorJson.java
+++ 
b/hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/prettyprint/LogicalOperatorPrettyPrintVisitorJson.java
@@ -171,6 +171,10 @@
     @Override
     public Void visitAggregateOperator(AggregateOperator op, Integer indent) 
throws AlgebricksException {
         addIndent(indent).append("\"operator\": \"aggregate\"");
+        if (!op.getExpressions().isEmpty()) {
+            addIndent(0).append(",\n");
+            pprintExprList(op.getExpressions(), indent);
+        }
         variablePrintHelper(op.getVariables(), indent);
         return null;
     }

--
To view, visit https://asterix-gerrit.ics.uci.edu/3384
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie4ba74d8455844e3913a4d4b5e2761b5ed23028b
Gerrit-Change-Number: 3384
Gerrit-PatchSet: 1
Gerrit-Owner: Till Westmann <ti...@apache.org>

Reply via email to