[ https://issues.apache.org/jira/browse/HBASE-18387?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16118415#comment-16118415 ]
Josh Elser commented on HBASE-18387: ------------------------------------ This LGTM. [~busbey] anything else to add? > [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 (v6.4.14#64029)