-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64282/
-----------------------------------------------------------

(Updated Dec. 6, 2017, 8:37 p.m.)


Review request for hive and Ashutosh Chauhan.


Changes
-------

This change addresses review comments


Bugs: HIVE-18173
    https://issues.apache.org/jira/browse/HIVE-18173


Repository: hive-git


Description
-------

Improve plans for correlated subqueries with non-equi predicate


Diffs (updated)
-----

  common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 75b77072c6 
  itests/hive-unit/pom.xml 3a435a8a52 
  
itests/hive-unit/src/test/java/org/apache/hive/jdbc/AbstractJdbcTriggersTest.java
 62ee66f717 
  
itests/hive-unit/src/test/java/org/apache/hive/jdbc/TestTriggersMoveWorkloadManager.java
 74ca958ea8 
  itests/src/test/resources/testconfiguration.properties cca1055fde 
  metastore/scripts/upgrade/mysql/046-HIVE-17566.mysql.sql 02288cbe42 
  metastore/scripts/upgrade/mysql/hive-schema-3.0.0.mysql.sql 915af8bf4b 
  ql/src/java/org/apache/hadoop/hive/ql/Context.java 6d48783d48 
  ql/src/java/org/apache/hadoop/hive/ql/Driver.java 4d52d748f1 
  ql/src/java/org/apache/hadoop/hive/ql/exec/tez/AmPluginNode.java eb6442180b 
  
ql/src/java/org/apache/hadoop/hive/ql/exec/tez/KillMoveTriggerActionHandler.java
 b16f1c30a0 
  ql/src/java/org/apache/hadoop/hive/ql/exec/tez/KillTriggerActionHandler.java 
50d234deaa 
  ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezSessionState.java 
dd879fc5e8 
  ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TezTask.java 8795cfcee1 
  ql/src/java/org/apache/hadoop/hive/ql/exec/tez/TriggerValidatorRunnable.java 
6414f05fe0 
  ql/src/java/org/apache/hadoop/hive/ql/exec/tez/WmEvent.java 33341ad4a9 
  ql/src/java/org/apache/hadoop/hive/ql/exec/tez/WmTezSession.java e78ef44c11 
  ql/src/java/org/apache/hadoop/hive/ql/exec/tez/WorkloadManager.java 
dbdbbf25db 
  ql/src/java/org/apache/hadoop/hive/ql/exec/tez/WorkloadManagerFederation.java 
9d56204240 
  ql/src/java/org/apache/hadoop/hive/ql/exec/tez/monitoring/PrintSummary.java 
8414c73e2b 
  ql/src/java/org/apache/hadoop/hive/ql/exec/tez/monitoring/TezJobMonitor.java 
9726af1506 
  ql/src/java/org/apache/hadoop/hive/ql/exec/vector/VectorizationContext.java 
0ad68166ae 
  
ql/src/java/org/apache/hadoop/hive/ql/exec/vector/expressions/CastStringToBoolean.java
 7a44035337 
  
ql/src/java/org/apache/hadoop/hive/ql/exec/vector/expressions/FuncStringToLong.java
 5c0a7fae56 
  
ql/src/java/org/apache/hadoop/hive/ql/exec/vector/expressions/VectorExpression.java
 8c2894b482 
  
ql/src/java/org/apache/hadoop/hive/ql/hooks/PostExecWMEventsSummaryPrinter.java 
83cca8903b 
  
ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/rules/HiveRelDecorrelator.java
 d1fe49c875 
  ql/src/java/org/apache/hadoop/hive/ql/parse/CalcitePlanner.java 76c82e2606 
  ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java 07742e0485 
  ql/src/java/org/apache/hadoop/hive/ql/udf/UDFToBoolean.java 57949d90aa 
  ql/src/java/org/apache/hadoop/hive/ql/wm/Trigger.java 4adad7a1b6 
  ql/src/java/org/apache/hadoop/hive/ql/wm/TriggerActionHandler.java 7995a8f639 
  ql/src/java/org/apache/hadoop/hive/ql/wm/TriggerContext.java PRE-CREATION 
  ql/src/java/org/apache/hadoop/hive/ql/wm/WmContext.java 7a7ef507e5 
  ql/src/test/org/apache/hadoop/hive/ql/exec/tez/TestWorkloadManager.java 
c58e4507f2 
  
ql/src/test/org/apache/hadoop/hive/ql/exec/vector/expressions/TestVectorMathFunctions.java
 e89f2e5a02 
  
ql/src/test/org/apache/hadoop/hive/ql/exec/vector/expressions/TestVectorTypeCasts.java
 6952b4598f 
  ql/src/test/queries/clientpositive/groupby_position.q 446b99d3cb 
  ql/src/test/queries/clientpositive/subquery_corr.q PRE-CREATION 
  ql/src/test/queries/clientpositive/subquery_in.q 7d4ece9dca 
  ql/src/test/queries/clientpositive/udf_to_boolean.q 1a50d055d5 
  ql/src/test/queries/clientpositive/vector_udf_string_to_boolean.q eeb5ab8819 
  ql/src/test/results/clientpositive/groupby_position.q.out 7351a06f9c 
  
ql/src/test/results/clientpositive/llap/insert_values_orig_table_use_metadata.q.out
 143742b3be 
  ql/src/test/results/clientpositive/llap/subquery_corr.q.out PRE-CREATION 
  ql/src/test/results/clientpositive/llap/subquery_exists.q.out dfe424046e 
  ql/src/test/results/clientpositive/llap/subquery_in.q.out 5dcdfdd15f 
  ql/src/test/results/clientpositive/llap/subquery_in_having.q.out 0ffbaaea34 
  ql/src/test/results/clientpositive/llap/subquery_notin.q.out 5da12584f0 
  ql/src/test/results/clientpositive/llap/vectorized_casts.q.out 84b4d9454d 
  ql/src/test/results/clientpositive/spark/groupby_position.q.out bcc512be09 
  ql/src/test/results/clientpositive/spark/spark_explainuser_1.q.out 6a4bea1bd4 
  ql/src/test/results/clientpositive/spark/subquery_exists.q.out fb13fb73e9 
  ql/src/test/results/clientpositive/spark/subquery_in.q.out e19240b7ca 
  ql/src/test/results/clientpositive/spark/subquery_multi.q.out a4282df08a 
  ql/src/test/results/clientpositive/spark/subquery_notin.q.out 0d12d0db60 
  ql/src/test/results/clientpositive/subquery_exists.q.out b6b31aaf47 
  ql/src/test/results/clientpositive/subquery_notexists.q.out a6175f8fec 
  ql/src/test/results/clientpositive/udf_to_boolean.q.out bee030ab92 
  ql/src/test/results/clientpositive/vector_empty_where.q.out 6b2c7fefa2 
  ql/src/test/results/clientpositive/vector_udf_string_to_boolean.q.out 
1b06698bca 
  ql/src/test/results/clientpositive/vectorized_casts.q.out f6f210533f 
  
serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/PrimitiveObjectInspectorUtils.java
 88e7fb00f4 
  
serde/src/test/org/apache/hadoop/hive/serde2/objectinspector/primitive/TestPrimitiveObjectInspectorUtils.java
 cbf19d0389 
  
standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/HiveAlterHandler.java
 13967d5186 
  
standalone-metastore/src/main/java/org/apache/hadoop/hive/metastore/ObjectStore.java
 2e80c9d3b1 


Diff: https://reviews.apache.org/r/64282/diff/5/

Changes: https://reviews.apache.org/r/64282/diff/4-5/


Testing
-------


Thanks,

Vineet Garg

Reply via email to