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

Nick Dimiduk commented on PHOENIX-2005:
---------------------------------------

And from myself:

{quote}
bq. Previously -z meant that you can only pass the zookeeper quorum (not port 
and znode parent from my understanding). If that is not the case, this logic 
seems correct. If that is the case, then why don't we still do the same thing 
by just setting the conf value and let the QueryUtil.getConnectionUrl() deal 
with constructing the JDBC string?

Thsi patch does simply set the conf value and let the 
QueryUtil.getConnectionUrl() deal with constructing the JDBC string. I'm using 
PhoenixDriver.ConnectionInfo object to parse the parameter provided by the user 
and using it's results to populate the conf object. The user-provided string is 
prepended with the jdbc bits because PhoenixDriver.ConnectionInfo.create() 
assumes it's working with a jdbc connection string. I can change it to handle 
either case, then there will be no need to prepend the user input with the 
"jdbc:...".
{quote}

> Connection utilities omit zk client port, parent znode
> ------------------------------------------------------
>
>                 Key: PHOENIX-2005
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-2005
>             Project: Phoenix
>          Issue Type: Bug
>            Reporter: Nick Dimiduk
>            Assignee: Nick Dimiduk
>
> Our config parsing utilities assume the zookeeper quorum server list is 
> sufficient for connecting to HBase. This not the case and should not be 
> assumed; take into account client port and znode parent. See conversation 
> over on PHOENIX-1980.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to