You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by Chen Xin YU <qi...@126.com> on 2015/11/24 03:45:17 UTC

Review Request 40627: HIVE-9600: add missing classes to hive-jdbc-standalone.jar

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

Review request for hive.


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


Repository: hive-git


Description
-------

hive-jdbc-standalone.jar does not have hadoop Configuration and maybe other hadoop-common classes required to open jdbc connection


Diffs
-----

  jdbc/pom.xml ea961a4 

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


Testing
-------

I tested hive-jdbc-standalone.jar with the patch, it works well with one jar for below scenarios:
1, common jdbc connection
2, jdbc connection with ssl
3, jdbc connection with http transport and ssl


Thanks,

Chen Xin YU


Re: Review Request 40627: HIVE-9600: add missing classes to hive-jdbc-standalone.jar

Posted by Chen Xin YU <qi...@126.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/40627/
-----------------------------------------------------------

(Updated Nov. 24, 2015, 2:59 a.m.)


Review request for hive.


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


Repository: hive-git


Description (updated)
-------

hive-jdbc-standalone.jar does not have hadoop Configuration and maybe other hadoop-common classes required to open jdbc connection
Exclude hadoop yarn and annotation jars instead of exclude all hadoop jars in jdbc/pom.xml
I tested hive-jdbc-standalone.jar with the patch, it works well with one jar for below scenarios:

1, common jdbc connection
2, jdbc connection with ssl
3, jdbc connection with http transport and ssl


Diffs
-----

  jdbc/pom.xml ea961a4 

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


Testing (updated)
-------


Thanks,

Chen Xin YU