Change in asterixdb[master]: [ASTERIXDB-3090][COMP] Enhance syntax and semantics of hash/broadcast...

[AsterixDB Code Review <do...@asterix-gerrit.ics.uci.edu>]

From Vijay Sarathy <vi...@couchbase.com>:

Attention is currently required from: Ali Alsuliman, Vijay Sarathy, murali4104@gmail.com, Till Westmann.
Hello Ali Alsuliman, Jenkins, Anon. E. Moose #1000171, murali4104@gmail.com, Till Westmann, 

I'd like you to reexamine a change. Please visit

    https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17280

to look at the new patch set (#6).

Change subject: [ASTERIXDB-3090][COMP] Enhance syntax and semantics of hash/broadcast join hints
......................................................................

[ASTERIXDB-3090][COMP] Enhance syntax and semantics of hash/broadcast join hints

- user model changes: yes
- storage format changes: no
- interface changes: no

Details:
This patch is to allow users to specify the build/probe side
of a hash join and the broadcast side of a broadcast hash join.

- Enhance the hash join hint "hashjoin" to specify the dataset
  with which to build/probe the hash table in the hash join.
  Example:
    A.field1 /*+ hashjoin build(B) */ = B.field2
    A.field1 /*+ hashjoin probe(B) */ = B.field2

- Enhance the broadcast join hint "hash-bcast" to specify
  the dataset to broadcast.
  Example:
    A.field1 /*+ hash-bcast(B) */ = B.field2

Change-Id: I67f3ea5bbd9dd4f5f4b836e765f42ccc1d7b53cc
---
A asterixdb/asterix-app/src/test/resources/optimizerts/queries/hints/broadcast_hint_4.sqlpp
M hyracks-fullstack/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/util/JoinUtils.java
M asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/tpch/q12_shipping_broadcast_ps.plan
M hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/expressions/HashJoinExpressionAnnotation.java
M hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/operators/logical/visitors/BroadcastSideSwitchingVisitor.java
M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/JoinEnum.java
A asterixdb/asterix-app/src/test/resources/optimizerts/results/hints/hashjoin_hint.plan
M asterixdb/asterix-app/src/test/resources/optimizerts/results_cbo/tpch/q12_shipping_broadcast.plan
M hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/expressions/BroadcastExpressionAnnotation.java
M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/cbo/JoinNode.java
A asterixdb/asterix-app/src/test/resources/optimizerts/results/hints/broadcast_hint_4.plan
M asterixdb/asterix-lang-sqlpp/src/main/javacc/SQLPP.jj
A asterixdb/asterix-app/src/test/resources/optimizerts/queries/hints/hashjoin_hint.sqlpp
13 files changed, 1,383 insertions(+), 109 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/80/17280/6
-- 
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17280
To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Change-Id: I67f3ea5bbd9dd4f5f4b836e765f42ccc1d7b53cc
Gerrit-Change-Number: 17280
Gerrit-PatchSet: 6
Gerrit-Owner: Vijay Sarathy <vi...@couchbase.com>
Gerrit-Reviewer: Ali Alsuliman <al...@gmail.com>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Till Westmann <ti...@couchbase.com>
Gerrit-Reviewer: Vijay Sarathy <vi...@couchbase.com>
Gerrit-Reviewer: murali4104@gmail.com
Gerrit-Attention: Ali Alsuliman <al...@gmail.com>
Gerrit-Attention: Vijay Sarathy <vi...@couchbase.com>
Gerrit-Attention: murali4104@gmail.com
Gerrit-Attention: Till Westmann <ti...@couchbase.com>
Gerrit-MessageType: newpatchset