hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Prasad Mujumdar" <pras...@cloudera.com>
Subject Re: Review Request 13262: HIVE-4573: JDBC Compliance getTables
Date Sun, 04 Aug 2013 10:06:47 GMT

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

(Updated Aug. 4, 2013, 10:06 a.m.)


Review request for hive.


Changes
-------

Changes per review comments


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


Repository: hive-git


Description
-------

Support alternate table type name for GetTables() and GetTableTypes().
Currently HiveServer2 supports Hive's native table type names eg. MANAGED_TABLE, VIRTUAL_VIEW
etc. The more common terms used by various DB vendors and tools are TABLE and VIEW. This makes
it difficult to integrate Hive with various tools or custom applications.
The patch introduces an alternate table type name mapping that can be configured at instance
level or session level.


Diffs (updated)
-----

  common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 555343e 
  conf/hive-default.xml.template f01e715 
  jdbc/src/test/org/apache/hive/jdbc/TestJdbcDriver2.java 1042125 
  service/src/java/org/apache/hive/service/cli/operation/ClassicTableTypeMapping.java PRE-CREATION

  service/src/java/org/apache/hive/service/cli/operation/GetTableTypesOperation.java eaf867e

  service/src/java/org/apache/hive/service/cli/operation/GetTablesOperation.java d9d0e9c 
  service/src/java/org/apache/hive/service/cli/operation/HiveTableTypeMapping.java PRE-CREATION

  service/src/java/org/apache/hive/service/cli/operation/TableTypeMapping.java PRE-CREATION

  service/src/java/org/apache/hive/service/cli/operation/TableTypeMappingFactory.java PRE-CREATION


Diff: https://reviews.apache.org/r/13262/diff/


Testing
-------

Added test cases in TestJdbcDriver2.


Thanks,

Prasad Mujumdar


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message