You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@asterixdb.apache.org by ti...@apache.org on 2019/05/08 23:22:08 UTC
[asterixdb] branch master updated: [NO ISSUE] show aggregate
functions in JSON plan
This is an automated email from the ASF dual-hosted git repository.
tillw pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/asterixdb.git
The following commit(s) were added to refs/heads/master by this push:
new a93a5c6 [NO ISSUE] show aggregate functions in JSON plan
a93a5c6 is described below
commit a93a5c6fe4f83cef973f5caaa540723c96e29ba2
Author: Till Westmann <ti...@apache.org>
AuthorDate: Wed May 8 13:29:05 2019 -0700
[NO ISSUE] show aggregate functions in JSON plan
- user model changes: more information in JSON plan
- storage format changes: no
- interface changes: no
Change-Id: Ie4ba74d8455844e3913a4d4b5e2761b5ed23028b
Reviewed-on: https://asterix-gerrit.ics.uci.edu/3384
Reviewed-by: Murtadha Hubail <mh...@apache.org>
Integration-Tests: Murtadha Hubail <mh...@apache.org>
Tested-by: Murtadha Hubail <mh...@apache.org>
---
.../algebra/prettyprint/LogicalOperatorPrettyPrintVisitorJson.java | 4 ++++
1 file changed, 4 insertions(+)
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 @@ public class LogicalOperatorPrettyPrintVisitorJson extends AbstractLogicalOperat
@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;
}