You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@asterixdb.apache.org by mb...@apache.org on 2023/02/15 17:21:31 UTC
[asterixdb] 01/07: [ASTERIXDB-3097][OTH] Print "build-side" only for hash join
This is an automated email from the ASF dual-hosted git repository.
mblow pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/asterixdb.git
commit f6e3615cb385feebc13086d3dec9eaea0ba1c59f
Author: Ali Alsuliman <al...@gmail.com>
AuthorDate: Thu Feb 2 20:20:53 2023 -0800
[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 <je...@fulliautomatix.ics.uci.edu>
Tested-by: Jenkins <je...@fulliautomatix.ics.uci.edu>
Reviewed-by: Murtadha Hubail <mh...@apache.org>
---
.../algebra/prettyprint/LogicalOperatorPrettyPrintVisitorJson.java | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
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 661e0cfad6..64a5e1a10b 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 @@ public class LogicalOperatorPrettyPrintVisitorJson extends AbstractLogicalOperat
}
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) {