>From Ali Alsuliman <[email protected]>:
Ali Alsuliman has uploaded this change for review. (
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
---
M
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/prettyprint/LogicalOperatorPrettyPrintVisitorJson.java
1 file changed, 20 insertions(+), 2 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/58/17358/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 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: 1
Gerrit-Owner: Ali Alsuliman <[email protected]>
Gerrit-MessageType: newchange
