[ 
https://issues.apache.org/jira/browse/HIVE-5659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13866045#comment-13866045
 ] 

Sushanth Sowmyan commented on HIVE-5659:
----------------------------------------

Also, there is an issue with the attached patch, I'm afraid what will happen 
with that merge is this:

There'll be a newConf created, with the new properties from hbase-default.xml , 
etc , overriding the properties in jobConf.

Then, with that merge direction, everything in newConf will override existing 
values in jobConf. Unless I'm reading HBaseConfiguration.merge incorrectly, I 
believe the goal here might have been to go the other way, but that'll give us 
a usable newConf, whereas we wanted to make modifications to jobConf. Since 
this is a duplicate of HIVE-6115, and that patch is closer to the required 
solution, I think, I'm going to shift over to that bug.


> HBaseStorageHandler overwrites Hive-set HBase properties with hbase-defaults
> ----------------------------------------------------------------------------
>
>                 Key: HIVE-5659
>                 URL: https://issues.apache.org/jira/browse/HIVE-5659
>             Project: Hive
>          Issue Type: Bug
>          Components: HBase Handler
>    Affects Versions: 0.12.0
>            Reporter: Craig Condit
>         Attachments: HIVE-5659.patch
>
>
> As part of the changes to HIVE-5260, it appears that HBase properties set in 
> hive-conf.xml are being clobbered by defaults from hbase-default.xml.
> Specifically, we noticed it when attempting to set hbase.zookeeper.quorum. 
> That value defaults to 'localhost' and results in queries of HBase tables 
> hanging attempting to acquire a lock from a Zookeeper instance which isn't 
> running.
> Any properties set in hive-site.xml will be overwritten by those in 
> hbase-default.xml, which doesn't seem good.
> The call to HBaseConfiguration.addHbaseResources(jobConf) seems to be the 
> culprit, around line 337.



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

Reply via email to