Steve Carlin has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/23784 )

Change subject: IMPALA-13574: Calcite Planner: Enable straight join hint
......................................................................

IMPALA-13574: Calcite Planner: Enable straight join hint

Enable the straight join feature within Calcite.

If the straight join keyword is found, it will avoid doing join
optimization on any joins within that query and all of its
subqueries.

Testing for this commit will be done in the Impala test
framework once it is enabled.

Change-Id: I7246f59f9077717e18f6a9d7155939f016c448a9
Reviewed-on: http://gerrit.cloudera.org:8080/23784
Reviewed-by: Jason Fehr <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>
Reviewed-by: Steve Carlin <[email protected]>
---
M java/calcite-planner/src/main/codegen/templates/Parser.jj
M 
java/calcite-planner/src/main/java/org/apache/impala/calcite/rel/node/ImpalaJoinRel.java
M 
java/calcite-planner/src/main/java/org/apache/impala/calcite/rules/ImpalaCoreRules.java
M 
java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteRelNodeConverter.java
M testdata/workloads/functional-query/queries/QueryTest/calcite.test
5 files changed, 82 insertions(+), 32 deletions(-)

Approvals:
  Jason Fehr: Looks good to me, but someone else must approve
  Impala Public Jenkins: Verified
  Steve Carlin: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/23784
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I7246f59f9077717e18f6a9d7155939f016c448a9
Gerrit-Change-Number: 23784
Gerrit-PatchSet: 10
Gerrit-Owner: Steve Carlin <[email protected]>
Gerrit-Reviewer: Aman Sinha <[email protected]>
Gerrit-Reviewer: Fang-Yu Rao <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Jason Fehr <[email protected]>
Gerrit-Reviewer: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Pranav Lodha <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>
Gerrit-Reviewer: Steve Carlin <[email protected]>

Reply via email to