phoenix-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (PHOENIX-831) First Connection to Phoenix blows up if it is tenant specific
Date Tue, 11 Mar 2014 07:32:43 GMT

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

ASF GitHub Bot commented on PHOENIX-831:
----------------------------------------

Github user JamesRTaylor commented on a diff in the pull request:

    https://github.com/apache/incubator-phoenix/pull/15#discussion_r10463896
  
    --- Diff: phoenix-core/src/main/java/org/apache/phoenix/jdbc/PhoenixConnection.java ---
    @@ -119,7 +116,36 @@ private static Properties newPropsWithSCN(long scn, Properties props)
{
             props.setProperty(PhoenixRuntime.CURRENT_SCN_ATTRIB, Long.toString(scn));
             return props;
         }
    -    
    --- End diff --
    
    Nice find. I'd rather keep things simple, though. I'm not worried about creating a new
Properties here. How about a util method that just instantiating a new, empty constructor
Properties and iterating through the entrySet of the one passed through to just add the values?
I'd rather do that then special case tenantId.



> First Connection to Phoenix blows up if it is tenant specific
> -------------------------------------------------------------
>
>                 Key: PHOENIX-831
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-831
>             Project: Phoenix
>          Issue Type: Bug
>    Affects Versions: 3.0.0
>            Reporter: Somesh Sasalatti
>
> Driver driver = DriverManager.getDriver(url);
> Properties props = new Properties();
> final String tenantId = "00Dxx0000001234";
> props.put(PhoenixRuntime.TENANT_ID_ATTRIB, tenantId);
> Connection connection = driver.connect(url, props);
> BOOM!
> I have pull request coming shortly



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Mime
View raw message