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

Review request for hive, Ashutosh Chauhan, Gopal V, and Jesús Camacho Rodríguez.


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


Repository: hive-git


Description
-------

- For queries that result in MR/Tez/Spark jobs on the cluster, save the 
temporary query results to a cache directory where they can be re-used.
- Add QueryResultsCache to manage cached results. Currently cache invalidation 
is time-based, update-based cache invalidation needs to be added later.
- Driver/SemanticAnalyzer/Calcite planner changes to lookup queries in the 
cache and use in place of the query plan.


Diffs
-----

  common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 05c2acd 
  common/src/java/org/apache/hadoop/hive/ql/log/PerfLogger.java 2767bca 
  itests/src/test/resources/testconfiguration.properties 1017249 
  itests/util/src/main/java/org/apache/hadoop/hive/ql/QTestUtil.java 3f377f9 
  ql/src/java/org/apache/hadoop/hive/ql/Driver.java 74595b0 
  ql/src/java/org/apache/hadoop/hive/ql/cache/results/CacheUsage.java 
PRE-CREATION 
  ql/src/java/org/apache/hadoop/hive/ql/cache/results/QueryResultsCache.java 
PRE-CREATION 
  ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java 2e1fd37 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/HiveCalciteUtil.java 
f0dd167 
  
ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/HiveRelOpMaterializationValidator.java
 PRE-CREATION 
  ql/src/java/org/apache/hadoop/hive/ql/parse/BaseSemanticAnalyzer.java 372cfad 
  ql/src/java/org/apache/hadoop/hive/ql/parse/CalcitePlanner.java 85a1f34 
  ql/src/java/org/apache/hadoop/hive/ql/parse/QBParseInfo.java ae2ec3d 
  ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java 28e1041 
  ql/src/java/org/apache/hadoop/hive/ql/plan/FetchWork.java 7243dc7 
  ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java 8af19d8 
  ql/src/test/queries/clientpositive/results_cache_1.q PRE-CREATION 
  ql/src/test/queries/clientpositive/results_cache_2.q PRE-CREATION 
  ql/src/test/queries/clientpositive/results_cache_capacity.q PRE-CREATION 
  ql/src/test/queries/clientpositive/results_cache_lifetime.q PRE-CREATION 
  ql/src/test/queries/clientpositive/results_cache_temptable.q PRE-CREATION 
  ql/src/test/queries/clientpositive/results_cache_with_masking.q PRE-CREATION 
  ql/src/test/results/clientpositive/llap/results_cache_1.q.out PRE-CREATION 
  ql/src/test/results/clientpositive/results_cache_1.q.out PRE-CREATION 
  ql/src/test/results/clientpositive/results_cache_2.q.out PRE-CREATION 
  ql/src/test/results/clientpositive/results_cache_capacity.q.out PRE-CREATION 
  ql/src/test/results/clientpositive/results_cache_lifetime.q.out PRE-CREATION 
  ql/src/test/results/clientpositive/results_cache_temptable.q.out PRE-CREATION 
  ql/src/test/results/clientpositive/results_cache_with_masking.q.out 
PRE-CREATION 


Diff: https://reviews.apache.org/r/65304/diff/1/


Testing
-------

qfile tests added.


Thanks,

Jason Dere

Reply via email to