Tamas Penzes commented on HBASE-18387:

I've simplified the parsing as suggested. :)

About named args: I think this way the parametrisation is more consistent, but 
I can be convinced about this too. ;-)

> [Thrift] Make principal configurable in DemoClient.java
> -------------------------------------------------------
>                 Key: HBASE-18387
>                 URL: https://issues.apache.org/jira/browse/HBASE-18387
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Lars George
>            Assignee: Tamas Penzes
>            Priority: Minor
>              Labels: beginner
>         Attachments: HBASE-18387.master.001.patch, 
> HBASE-18387.master.002.patch, HBASE-18387.master.003.patch
> In the Thrift1 demo client we have this code:
> {code}
> transport = new TSaslClientTransport("GSSAPI", null,
>   "hbase", // Thrift server user name, should be an authorized proxy user.
>   host, // Thrift server domain
>   saslProperties, null, transport);
> {code}
> This will only work when the Thrift server is started with the {{hbase}} 
> principal. Often this may deviate, for example I am using {{hbase-thrift}} to 
> separate the names from those of backend servers. 
> What we need is either an additional command line option to specify the name, 
> or a property that can be set with -D and can be passed at runtime. I prefer 
> the former, as the latter is making this a little convoluted.

This message was sent by Atlassian JIRA

Reply via email to