>From Ali Alsuliman <[email protected]>: Ali Alsuliman has submitted this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17358 )
Change subject: [ASTERIXDB-3097][OTH] Print "build-side" only for hash join ...................................................................... [ASTERIXDB-3097][OTH] Print "build-side" only for hash join - user model changes: no - storage format changes: no - interface changes: no Details: Fix the JSON plan printer to only print the "build-side" for hash join. Change-Id: Ic418905d45fa9edb7c77e5ccb748d07bbd0d8c64 Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17358 Integration-Tests: Jenkins <[email protected]> Tested-by: Jenkins <[email protected]> Reviewed-by: Murtadha Hubail <[email protected]> --- M hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/prettyprint/LogicalOperatorPrettyPrintVisitorJson.java 1 file changed, 24 insertions(+), 2 deletions(-) Approvals: Murtadha Hubail: Looks good to me, approved Jenkins: Verified; Verified Anon. E. Moose #1000171: 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 661e0cf..64a5e1a 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 @@ -954,8 +954,9 @@ } private void writeBuildSide(AbstractBinaryJoinOperator op) throws IOException { - int buildInputIndex = printInputsInReverse(op) ? 0 : 1; - jsonGenerator.writeNumberField("build-side", buildInputIndex); + if (isHashJoin(op)) { + jsonGenerator.writeNumberField("build-side", 0); + } } private static boolean printInputsInReverse(AbstractLogicalOperator op) { -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17358 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: neo Gerrit-Change-Id: Ic418905d45fa9edb7c77e5ccb748d07bbd0d8c64 Gerrit-Change-Number: 17358 Gerrit-PatchSet: 2 Gerrit-Owner: Ali Alsuliman <[email protected]> Gerrit-Reviewer: Ali Alsuliman <[email protected]> Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Murtadha Hubail <[email protected]> Gerrit-MessageType: merged
