hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Prasad Mujumdar" <pras...@cloudera.com>
Subject Review Request: HIVE-3166: The Hive JDBC driver should accept hive conf and hive variables via connection URL
Date Fri, 22 Jun 2012 02:48:36 GMT

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

Review request for hive, Carl Steinbach and Carl Steinbach.


Description
-------

Support passing configuration and substitution variable as part of JDBC connection string.

The new format of the URL is jdbc:hive://<host>:<port>/dbName?hive_conf_list#hive_var_list
  , where the optional conf and var lists are semicolon separated <key>=<val>
pairs. As before, if the host/port is not specified, it the driver runs an embedded hive.
examples -
jdbc:hive://ubuntu:11000/db2?hive.cli.conf.printheader=true;hive.exec.mode.local.auto.inputbytes.max=9999#stab=salesTable;icol=customerID
jdbc:hive://?hive.cli.conf.printheader=true;hive.exec.mode.local.auto.inputbytes.max=9999#stab=salesTable;icol=customerID

The patch include new routines to parse the URL. The conf values are added to HiveConf when
hive is running in embedded mode otherwise they are configured using 'set' statement. The
variable substitution is only used in case of embedded mode.


This addresses bug HIVE-3166.
    https://issues.apache.org/jira/browse/HIVE-3166


Diffs
-----

  jdbc/src/java/org/apache/hadoop/hive/jdbc/HiveConnection.java 6618243 
  jdbc/src/java/org/apache/hadoop/hive/jdbc/HiveDriver.java c61425f 
  jdbc/src/java/org/apache/hadoop/hive/jdbc/Utils.java 24d5882 
  jdbc/src/test/org/apache/hadoop/hive/jdbc/TestJdbcDriver.java f6a904f 

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


Testing
-------

Ran JDBC tests.
Added new test case for the extended URL format.


Thanks,

Prasad Mujumdar


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