Murtadha Hubail has submitted this change and it was merged. Change subject: [ASTERIXDB-2510][OTH] Fix Forward Operator JSON Plan ......................................................................
[ASTERIXDB-2510][OTH] Fix Forward Operator JSON Plan - user model changes: no - storage format changes: no - interface changes: no Details: - Add separator in forward operator name and expression JSON fields. Change-Id: Ibbffeebe82fff8e5282ca3c16343f9c909d35262 Reviewed-on: https://asterix-gerrit.ics.uci.edu/3117 Reviewed-by: Ali Alsuliman <ali.al.solai...@gmail.com> Sonar-Qube: Jenkins <jenk...@fulliautomatix.ics.uci.edu> Tested-by: Jenkins <jenk...@fulliautomatix.ics.uci.edu> Integration-Tests: Jenkins <jenk...@fulliautomatix.ics.uci.edu> Contrib: Jenkins <jenk...@fulliautomatix.ics.uci.edu> --- M hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/prettyprint/LogicalOperatorPrettyPrintVisitorJson.java 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Ali Alsuliman: Looks good to me, approved Jenkins: Verified; No violations found; ; Verified Objections: Anon. E. Moose #1000171: Violations found 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 8ba2aff..f502d14 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 @@ -633,6 +633,7 @@ @Override public Void visitForwardOperator(ForwardOperator op, Integer indent) throws AlgebricksException { addIndent(indent).append("\"operator\": \"forward\""); + addIndent(0).append(",\n"); addIndent(indent).append("\"expressions\": \"" + op.getRangeMapExpression().getValue().accept(exprVisitor, indent).replace('"', ' ') + "\""); return null; -- To view, visit https://asterix-gerrit.ics.uci.edu/3117 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibbffeebe82fff8e5282ca3c16343f9c909d35262 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha Hubail <mhub...@apache.org> Gerrit-Reviewer: Ali Alsuliman <ali.al.solai...@gmail.com> 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>