Murtadha Hubail has submitted this change and it was merged.

Change subject: [ASTERIXDB-2377][OTH] Fix JSON of Additional Expressions
......................................................................


[ASTERIXDB-2377][OTH] Fix JSON of Additional Expressions

- user model changes: no
- storage format changes: no
- interface changes: no

Details:
- Fix JSON of additional expressions in the JSON
  plan printer of InsertDeleteUpsertOperator.

Change-Id: I2d13b91b4e5e36b156f52fbb09f0ddcffe1deda3
Reviewed-on: https://asterix-gerrit.ics.uci.edu/2627
Sonar-Qube: Jenkins <jenk...@fulliautomatix.ics.uci.edu>
Tested-by: Jenkins <jenk...@fulliautomatix.ics.uci.edu>
Contrib: Jenkins <jenk...@fulliautomatix.ics.uci.edu>
Integration-Tests: Jenkins <jenk...@fulliautomatix.ics.uci.edu>
Reviewed-by: Till Westmann <ti...@apache.org>
---
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/prettyprint/LogicalOperatorPrettyPrintVisitorJson.java
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Anon. E. Moose #1000171: 
  Till Westmann: Looks good to me, approved
  Jenkins: Verified; No violations found; ; Verified



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 1076eb5..8acf08b 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
@@ -544,9 +544,9 @@
         addIndent(indent).append("\"from-record\": \"")
                 
.append(op.getPayloadExpression().getValue().accept(exprVisitor, indent) + 
"\"");
         if (op.getAdditionalNonFilteringExpressions() != null) {
-            buffer.append(",\n\"meta\": \"");
+            buffer.append(",\n\"meta\": {");
             pprintExprList(op.getAdditionalNonFilteringExpressions(), 0);
-            buffer.append("\"");
+            buffer.append("}");
         }
         buffer.append(",\n");
         addIndent(indent).append("\"partitioned-by\": {");

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I2d13b91b4e5e36b156f52fbb09f0ddcffe1deda3
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: release-0.9.4-pre-rc
Gerrit-Owner: Murtadha Hubail <mhub...@apache.org>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <jenk...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Murtadha Hubail <mhub...@apache.org>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>

Reply via email to