-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51593/
-----------------------------------------------------------
(Updated Sept. 30, 2016, 10:56 p.m.)
Review request for hive, Mohit Sabharwal, Sergio Pena, and Szehon Ho.
Changes
-------
Fixed HiveQA test failures and refactored some of the methods to improve
readability
Bugs: HIVE-14063
https://issues.apache.org/jira/browse/HIVE-14063
Repository: hive-git
Description
-------
This change adds a new optional configuration file for Beeline. If this file is
present at predefined locations, Beeline will attempt to create the connection
url using the hive-site.xml found in classpath and another user-specific
configuration file. Beeline then connects automatically using the url generated
based on these configuration files. The main objective of the change is to
provide user another way to connect to the HiveServer2 without providing the
connection url everytime. The configuration file uses hadoop xml format so that
we can support encryption/obfuscation using hadoop credential manager API in
the future.
Properties in the user-specific configuration file override the properties
derived from hive-site.xml.
Tested using newly added unit tests and itests in various Hiveserver2
configurations.
Diffs (updated)
-----
beeline/src/java/org/apache/hive/beeline/BeeLine.java
79922d267eaf13216e8a8e4826c050d334266579
beeline/src/java/org/apache/hive/beeline/Commands.java
039e3549209682b5ebdde937607b77c3ec094527
beeline/src/java/org/apache/hive/beeline/hs2connection/BeelineHS2ConnectionFileParseException.java
PRE-CREATION
beeline/src/java/org/apache/hive/beeline/hs2connection/HS2ConnectionFileParser.java
PRE-CREATION
beeline/src/java/org/apache/hive/beeline/hs2connection/HS2ConnectionFileUtils.java
PRE-CREATION
beeline/src/java/org/apache/hive/beeline/hs2connection/HiveSiteHS2ConnectionFileParser.java
PRE-CREATION
beeline/src/java/org/apache/hive/beeline/hs2connection/UserHS2ConnectionFileParser.java
PRE-CREATION
beeline/src/test/org/apache/hive/beeline/TestBeeLineHistory.java
e86197626a76474c189b683fa55f02959c7a8bff
beeline/src/test/org/apache/hive/beeline/hs2connection/TestUserHS2ConnectionFileParser.java
PRE-CREATION
beeline/src/test/resources/hive-site.xml
5f310d68245275ac9dc24df45579784019eea332
beeline/src/test/resources/test-hs2-conn-conf-kerberos-http.xml PRE-CREATION
beeline/src/test/resources/test-hs2-conn-conf-kerberos-nossl.xml PRE-CREATION
beeline/src/test/resources/test-hs2-conn-conf-kerberos-ssl.xml PRE-CREATION
beeline/src/test/resources/test-hs2-connection-conf-list.xml PRE-CREATION
beeline/src/test/resources/test-hs2-connection-config-noauth.xml PRE-CREATION
beeline/src/test/resources/test-hs2-connection-multi-conf-list.xml
PRE-CREATION
beeline/src/test/resources/test-hs2-connection-zookeeper-config.xml
PRE-CREATION
itests/hive-unit/src/test/java/org/apache/hive/beeline/hs2connection/TestBeelineConnectionUsingHiveSite.java
PRE-CREATION
itests/hive-unit/src/test/java/org/apache/hive/beeline/hs2connection/TestBeelineWithHS2ConnectionFile.java
PRE-CREATION
itests/hive-unit/src/test/java/org/apache/hive/beeline/hs2connection/TestBeelineWithUserHs2ConnectionFile.java
PRE-CREATION
Diff: https://reviews.apache.org/r/51593/diff/
Testing
-------
Thanks,
Vihang Karajgaonkar
