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

(Updated July 25, 2019, 5:35 p.m.)


Review request for hive.


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


Repository: hive-git


Description
-------

This patch adds an initial integration for Apache Kudu backed tables
by supporting the creation of external tables pointed at existing
underlying Kudu tables.

SELECT queries can read from the tables including pushing most
predicates/filters into the Kudu scanners. Future work should
complete support for Kudu predicates.

INSERT queries can write to the tables. However, they currently
use Kudu UPSERT operations when writing. Future work should
complete support for INSERT, UPDATE, and DELETE.

Note: The table properties and class names match the values used by
Apache Impala when creating HMS entries for Kudu tables. This
means tables created by Impala can be used by Hive and vice versa.


Diffs (updated)
-----

  common/src/java/org/apache/hadoop/hive/conf/HiveConf.java a9fcf37e2e 
  itests/pom.xml 345e9220df 
  itests/qtest-kudu/pom.xml PRE-CREATION 
  
itests/qtest-kudu/src/test/java/org/apache/hadoop/hive/cli/TestKuduCliDriver.java
 PRE-CREATION 
  
itests/qtest-kudu/src/test/java/org/apache/hadoop/hive/cli/TestKuduNegativeCliDriver.java
 PRE-CREATION 
  itests/qtest-kudu/src/test/java/org/apache/hadoop/hive/cli/package-info.java 
PRE-CREATION 
  itests/util/pom.xml 607fd4724e 
  itests/util/src/main/java/org/apache/hadoop/hive/cli/control/CliConfigs.java 
5c17e1ade6 
  
itests/util/src/main/java/org/apache/hadoop/hive/cli/control/CoreKuduCliDriver.java
 PRE-CREATION 
  
itests/util/src/main/java/org/apache/hadoop/hive/cli/control/CoreKuduNegativeCliDriver.java
 PRE-CREATION 
  itests/util/src/main/java/org/apache/hadoop/hive/kudu/KuduTestSetup.java 
PRE-CREATION 
  itests/util/src/main/java/org/apache/hadoop/hive/kudu/package-info.java 
PRE-CREATION 
  itests/util/src/main/java/org/apache/hadoop/hive/ql/QTestMiniClusters.java 
bd4c76ed66 
  kudu-handler/pom.xml PRE-CREATION 
  kudu-handler/src/java/org/apache/hadoop/hive/kudu/KuduHiveUtils.java 
PRE-CREATION 
  kudu-handler/src/java/org/apache/hadoop/hive/kudu/KuduInputFormat.java 
PRE-CREATION 
  kudu-handler/src/java/org/apache/hadoop/hive/kudu/KuduOutputFormat.java 
PRE-CREATION 
  kudu-handler/src/java/org/apache/hadoop/hive/kudu/KuduPredicateHandler.java 
PRE-CREATION 
  kudu-handler/src/java/org/apache/hadoop/hive/kudu/KuduSerDe.java PRE-CREATION 
  kudu-handler/src/java/org/apache/hadoop/hive/kudu/KuduStorageHandler.java 
PRE-CREATION 
  kudu-handler/src/java/org/apache/hadoop/hive/kudu/KuduWritable.java 
PRE-CREATION 
  kudu-handler/src/java/org/apache/hadoop/hive/kudu/package-info.java 
PRE-CREATION 
  kudu-handler/src/test/org/apache/hadoop/hive/kudu/KuduTestUtils.java 
PRE-CREATION 
  kudu-handler/src/test/org/apache/hadoop/hive/kudu/TestKuduInputFormat.java 
PRE-CREATION 
  kudu-handler/src/test/org/apache/hadoop/hive/kudu/TestKuduOutputFormat.java 
PRE-CREATION 
  
kudu-handler/src/test/org/apache/hadoop/hive/kudu/TestKuduPredicateHandler.java 
PRE-CREATION 
  kudu-handler/src/test/org/apache/hadoop/hive/kudu/TestKuduSerDe.java 
PRE-CREATION 
  kudu-handler/src/test/org/apache/hadoop/hive/kudu/package-info.java 
PRE-CREATION 
  kudu-handler/src/test/queries/negative/kudu_config.q PRE-CREATION 
  kudu-handler/src/test/queries/positive/kudu_complex_queries.q PRE-CREATION 
  kudu-handler/src/test/queries/positive/kudu_queries.q PRE-CREATION 
  kudu-handler/src/test/results/negative/kudu_config.q.out PRE-CREATION 
  kudu-handler/src/test/results/positive/kudu_complex_queries.q.out 
PRE-CREATION 
  kudu-handler/src/test/results/positive/kudu_queries.q.out PRE-CREATION 
  
llap-server/src/java/org/apache/hadoop/hive/llap/cli/service/AsyncTaskCopyAuxJars.java
 7b2e32bea2 
  pom.xml 4549d4878c 
  ql/src/java/org/apache/hadoop/hive/ql/io/sarg/ConvertAstToSearchArg.java 
27fe828b75 


Diff: https://reviews.apache.org/r/71133/diff/6/

Changes: https://reviews.apache.org/r/71133/diff/5-6/


Testing
-------


Thanks,

Grant Henke

Reply via email to