[jira] [Commented] (HBASE-18387) [Thrift] Make principal configurable in DemoClient.java

2017-08-17 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-18387?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16131126#comment-16131126
 ] 

Hudson commented on HBASE-18387:


FAILURE: Integrated in Jenkins build HBASE-14070.HLC #233 (See 
[https://builds.apache.org/job/HBASE-14070.HLC/233/])
HBASE-18387: [Thrift] Make principal configurable in DemoClient.java (elserj: 
rev 8da77b414657f0ee3b093913de5f92eba17ecd2a)
* (edit) hbase-examples/README.txt
* (edit) 
hbase-examples/src/main/java/org/apache/hadoop/hbase/thrift/DemoClient.java


> [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
> Fix For: 2.0.0, 1.4.0, 1.3.2, 1.2.7, 1.1.12
>
> 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)


[jira] [Commented] (HBASE-18387) [Thrift] Make principal configurable in DemoClient.java

2017-08-14 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-18387?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16126107#comment-16126107
 ] 

Hudson commented on HBASE-18387:


FAILURE: Integrated in Jenkins build HBase-1.4 #854 (See 
[https://builds.apache.org/job/HBase-1.4/854/])
HBASE-18387: [Thrift] Make principal configurable in DemoClient.java (apurtell: 
rev 2d06c1f7ba65f5a153adc726cc9c5cd9805bac5d)
* (edit) 
hbase-examples/src/main/java/org/apache/hadoop/hbase/thrift/DemoClient.java
* (edit) hbase-examples/README.txt


> [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
> Fix For: 2.0.0, 1.4.0, 1.3.2, 1.2.7, 1.1.12
>
> 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)


[jira] [Commented] (HBASE-18387) [Thrift] Make principal configurable in DemoClient.java

2017-08-11 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-18387?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16123084#comment-16123084
 ] 

Hudson commented on HBASE-18387:


FAILURE: Integrated in Jenkins build HBase-Trunk_matrix #3511 (See 
[https://builds.apache.org/job/HBase-Trunk_matrix/3511/])
HBASE-18387: [Thrift] Make principal configurable in DemoClient.java (elserj: 
rev 8da77b414657f0ee3b093913de5f92eba17ecd2a)
* (edit) hbase-examples/README.txt
* (edit) 
hbase-examples/src/main/java/org/apache/hadoop/hbase/thrift/DemoClient.java


> [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
> Fix For: 2.0.0, 1.4.0, 1.3.2, 1.2.7, 1.1.12
>
> 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)


[jira] [Commented] (HBASE-18387) [Thrift] Make principal configurable in DemoClient.java

2017-08-11 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-18387?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16123016#comment-16123016
 ] 

Hudson commented on HBASE-18387:


FAILURE: Integrated in Jenkins build HBase-1.4 #847 (See 
[https://builds.apache.org/job/HBase-1.4/847/])
HBASE-18387: [Thrift] Make principal configurable in DemoClient.java (elserj: 
rev d72ba890d9dedd7a58d5d36a54d31b47dcd6d035)
* (edit) hbase-examples/README.txt
* (edit) 
hbase-examples/src/main/java/org/apache/hadoop/hbase/thrift/DemoClient.java


> [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
> Fix For: 2.0.0, 1.4.0, 1.3.2, 1.2.7, 1.1.12
>
> 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)


[jira] [Commented] (HBASE-18387) [Thrift] Make principal configurable in DemoClient.java

2017-08-11 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-18387?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16123009#comment-16123009
 ] 

Hudson commented on HBASE-18387:


FAILURE: Integrated in Jenkins build HBase-1.3-JDK8 #242 (See 
[https://builds.apache.org/job/HBase-1.3-JDK8/242/])
HBASE-18387: [Thrift] Make principal configurable in DemoClient.java (elserj: 
rev a2a28a7806549b3dd337163432ad12d944add634)
* (edit) 
hbase-examples/src/main/java/org/apache/hadoop/hbase/thrift/DemoClient.java
* (edit) hbase-examples/README.txt


> [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
> Fix For: 2.0.0, 1.4.0, 1.3.2, 1.2.7, 1.1.12
>
> 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)


[jira] [Commented] (HBASE-18387) [Thrift] Make principal configurable in DemoClient.java

2017-08-11 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-18387?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16122987#comment-16122987
 ] 

Hudson commented on HBASE-18387:


FAILURE: Integrated in Jenkins build HBase-1.2-JDK8 #186 (See 
[https://builds.apache.org/job/HBase-1.2-JDK8/186/])
HBASE-18387: [Thrift] Make principal configurable in DemoClient.java (elserj: 
rev a732b67ce8c7c08f867807ce7f1b0b47352a7d5e)
* (edit) hbase-examples/README.txt
* (edit) 
hbase-examples/src/main/java/org/apache/hadoop/hbase/thrift/DemoClient.java


> [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
> Fix For: 2.0.0, 1.4.0, 1.3.2, 1.2.7, 1.1.12
>
> 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)


[jira] [Commented] (HBASE-18387) [Thrift] Make principal configurable in DemoClient.java

2017-08-11 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-18387?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16122918#comment-16122918
 ] 

Hudson commented on HBASE-18387:


SUCCESS: Integrated in Jenkins build HBase-1.1-JDK8 #1983 (See 
[https://builds.apache.org/job/HBase-1.1-JDK8/1983/])
HBASE-18387: [Thrift] Make principal configurable in DemoClient.java (elserj: 
rev ebc1d2bf2c52bbaf5e2bc81afab6298558978239)
* (edit) hbase-examples/README.txt
* (edit) 
hbase-examples/src/main/java/org/apache/hadoop/hbase/thrift/DemoClient.java


> [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
> Fix For: 2.0.0, 1.4.0, 1.3.2, 1.2.7, 1.1.12
>
> 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)


[jira] [Commented] (HBASE-18387) [Thrift] Make principal configurable in DemoClient.java

2017-08-10 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-18387?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16122892#comment-16122892
 ] 

Hudson commented on HBASE-18387:


FAILURE: Integrated in Jenkins build HBase-2.0 #308 (See 
[https://builds.apache.org/job/HBase-2.0/308/])
HBASE-18387: [Thrift] Make principal configurable in DemoClient.java (elserj: 
rev ee15c2c296d49e943c61b99e851449ee94394621)
* (edit) 
hbase-examples/src/main/java/org/apache/hadoop/hbase/thrift/DemoClient.java
* (edit) hbase-examples/README.txt


> [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
> Fix For: 2.0.0, 1.4.0, 1.3.2, 1.2.7, 1.1.12
>
> 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)


[jira] [Commented] (HBASE-18387) [Thrift] Make principal configurable in DemoClient.java

2017-08-10 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-18387?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16122874#comment-16122874
 ] 

Hudson commented on HBASE-18387:


FAILURE: Integrated in Jenkins build HBase-1.1-JDK7 #1899 (See 
[https://builds.apache.org/job/HBase-1.1-JDK7/1899/])
HBASE-18387: [Thrift] Make principal configurable in DemoClient.java (elserj: 
rev ebc1d2bf2c52bbaf5e2bc81afab6298558978239)
* (edit) hbase-examples/README.txt
* (edit) 
hbase-examples/src/main/java/org/apache/hadoop/hbase/thrift/DemoClient.java


> [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
> Fix For: 2.0.0, 1.4.0, 1.3.2, 1.2.7, 1.1.12
>
> 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)


[jira] [Commented] (HBASE-18387) [Thrift] Make principal configurable in DemoClient.java

2017-08-10 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-18387?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16122816#comment-16122816
 ] 

Hudson commented on HBASE-18387:


SUCCESS: Integrated in Jenkins build HBase-1.2-IT #926 (See 
[https://builds.apache.org/job/HBase-1.2-IT/926/])
HBASE-18387: [Thrift] Make principal configurable in DemoClient.java (elserj: 
rev a732b67ce8c7c08f867807ce7f1b0b47352a7d5e)
* (edit) 
hbase-examples/src/main/java/org/apache/hadoop/hbase/thrift/DemoClient.java
* (edit) hbase-examples/README.txt


> [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
> Fix For: 2.0.0, 1.4.0, 1.3.2, 1.2.7, 1.1.12
>
> 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)


[jira] [Commented] (HBASE-18387) [Thrift] Make principal configurable in DemoClient.java

2017-08-10 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-18387?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16122807#comment-16122807
 ] 

Hudson commented on HBASE-18387:


SUCCESS: Integrated in Jenkins build HBase-1.3-IT #165 (See 
[https://builds.apache.org/job/HBase-1.3-IT/165/])
HBASE-18387: [Thrift] Make principal configurable in DemoClient.java (elserj: 
rev a2a28a7806549b3dd337163432ad12d944add634)
* (edit) hbase-examples/README.txt
* (edit) 
hbase-examples/src/main/java/org/apache/hadoop/hbase/thrift/DemoClient.java


> [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
> Fix For: 2.0.0, 1.4.0, 1.3.2, 1.2.7, 1.1.12
>
> 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)


[jira] [Commented] (HBASE-18387) [Thrift] Make principal configurable in DemoClient.java

2017-08-10 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-18387?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16122801#comment-16122801
 ] 

Hudson commented on HBASE-18387:


FAILURE: Integrated in Jenkins build HBase-1.3-JDK7 #229 (See 
[https://builds.apache.org/job/HBase-1.3-JDK7/229/])
HBASE-18387: [Thrift] Make principal configurable in DemoClient.java (elserj: 
rev a2a28a7806549b3dd337163432ad12d944add634)
* (edit) 
hbase-examples/src/main/java/org/apache/hadoop/hbase/thrift/DemoClient.java
* (edit) hbase-examples/README.txt


> [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
> Fix For: 2.0.0, 1.4.0, 1.3.2, 1.2.7, 1.1.12
>
> 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)


[jira] [Commented] (HBASE-18387) [Thrift] Make principal configurable in DemoClient.java

2017-08-10 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-18387?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16122800#comment-16122800
 ] 

Hudson commented on HBASE-18387:


FAILURE: Integrated in Jenkins build HBase-1.2-JDK7 #191 (See 
[https://builds.apache.org/job/HBase-1.2-JDK7/191/])
HBASE-18387: [Thrift] Make principal configurable in DemoClient.java (elserj: 
rev a732b67ce8c7c08f867807ce7f1b0b47352a7d5e)
* (edit) 
hbase-examples/src/main/java/org/apache/hadoop/hbase/thrift/DemoClient.java
* (edit) hbase-examples/README.txt


> [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
> Fix For: 2.0.0, 1.4.0, 1.3.2, 1.2.7, 1.1.12
>
> 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)


[jira] [Commented] (HBASE-18387) [Thrift] Make principal configurable in DemoClient.java

2017-08-10 Thread Sean Busbey (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-18387?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16122256#comment-16122256
 ] 

Sean Busbey commented on HBASE-18387:
-

+1

> [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)


[jira] [Commented] (HBASE-18387) [Thrift] Make principal configurable in DemoClient.java

2017-08-08 Thread Josh Elser (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-18387?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=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)


[jira] [Commented] (HBASE-18387) [Thrift] Make principal configurable in DemoClient.java

2017-08-08 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-18387?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16118273#comment-16118273
 ] 

Hadoop QA commented on HBASE-18387:
---

| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue}  0m 
20s{color} | {color:blue} Docker mode activated. {color} |
| {color:green}+1{color} | {color:green} hbaseanti {color} | {color:green}  0m  
0s{color} | {color:green} Patch does not have any anti-patterns. {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green}  0m  
0s{color} | {color:green} The patch does not contain any @author tags. {color} |
| {color:red}-1{color} | {color:red} test4tests {color} | {color:red}  0m  
0s{color} | {color:red} The patch doesn't appear to include any new or modified 
tests. Please justify why no new tests are needed for this patch. Also please 
list what manual steps were performed to verify this patch. {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  4m 
16s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  0m 
18s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
14s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  0m 
18s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  0m 
38s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
16s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  0m 
23s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  0m 
18s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  0m 
18s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
16s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  0m 
17s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green}  0m 
 0s{color} | {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} hadoopcheck {color} | {color:green} 
33m 11s{color} | {color:green} Patch does not cause any errors with Hadoop 
2.6.1 2.6.2 2.6.3 2.6.4 2.6.5 2.7.1 2.7.2 2.7.3 or 3.0.0-alpha4. {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  0m 
34s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
13s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
40s{color} | {color:green} hbase-examples in the patch passed. {color} |
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green}  0m 
 7s{color} | {color:green} The patch does not generate ASF License warnings. 
{color} |
| {color:black}{color} | {color:black} {color} | {color:black} 42m 41s{color} | 
{color:black} {color} |
\\
\\
|| Subsystem || Report/Notes ||
| Docker | Client=1.12.3 Server=1.12.3 Image:yetus/hbase:bdc94b1 |
| JIRA Issue | HBASE-18387 |
| JIRA Patch URL | 
https://issues.apache.org/jira/secure/attachment/12880814/HBASE-18387.master.003.patch
 |
| Optional Tests |  asflicense  javac  javadoc  unit  findbugs  hadoopcheck  
hbaseanti  checkstyle  compile  |
| uname | Linux 11b9d15e57e5 3.13.0-119-generic #166-Ubuntu SMP Wed May 3 
12:18:55 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | 
/home/jenkins/jenkins-slave/workspace/PreCommit-HBASE-Build@2/component/dev-support/hbase-personality.sh
 |
| git revision | master / 0339068 |
| Default Java | 1.8.0_131 |
| findbugs | v3.1.0-RC3 |
|  Test Results | 
https://builds.apache.org/job/PreCommit-HBASE-Build/7977/testReport/ |
| modules | C: hbase-examples U: hbase-examples |
| Console output | 
https://builds.apache.org/job/PreCommit-HBASE-Build/7977/console |
| Powered by | Apache Yetus 0.4.0   http://yetus.apache.org |


This message was automatically generated.



> [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
>

[jira] [Commented] (HBASE-18387) [Thrift] Make principal configurable in DemoClient.java

2017-08-08 Thread Tamas Penzes (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-18387?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16118231#comment-16118231
 ] 

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
(v6.4.14#64029)


[jira] [Commented] (HBASE-18387) [Thrift] Make principal configurable in DemoClient.java

2017-08-07 Thread Josh Elser (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-18387?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16117298#comment-16117298
 ] 

Josh Elser commented on HBASE-18387:


{code}
+System.out.println("Usage: DemoClient host port [secure=false] 
[server-principal=hbase]");
{code}

I think the parsing could be simplified if instead the arguments were 
{{DemoClient host port \[secure=false \[server-principal=hbase\] \]}} (the 
server principal option can only be provided if the secure option is also 
provided). Trying to do optional-positional arguments in the manner you tried 
to implement is super confusing, IMO (named args are a bit more clear in that 
regard, but I digress).

Any chance I could convince you to tweak this? :)

> [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
>
>
> 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)


[jira] [Commented] (HBASE-18387) [Thrift] Make principal configurable in DemoClient.java

2017-08-07 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-18387?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16116410#comment-16116410
 ] 

Hadoop QA commented on HBASE-18387:
---

| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue}  0m 
16s{color} | {color:blue} Docker mode activated. {color} |
| {color:green}+1{color} | {color:green} hbaseanti {color} | {color:green}  0m  
0s{color} | {color:green} Patch does not have any anti-patterns. {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green}  0m  
0s{color} | {color:green} The patch does not contain any @author tags. {color} |
| {color:red}-1{color} | {color:red} test4tests {color} | {color:red}  0m  
0s{color} | {color:red} The patch doesn't appear to include any new or modified 
tests. Please justify why no new tests are needed for this patch. Also please 
list what manual steps were performed to verify this patch. {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  3m 
32s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  0m 
16s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
12s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  0m 
15s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  0m 
25s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
12s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  0m 
16s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  0m 
16s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  0m 
16s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
12s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  0m 
14s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green}  0m 
 0s{color} | {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} hadoopcheck {color} | {color:green} 
32m  3s{color} | {color:green} Patch does not cause any errors with Hadoop 
2.6.1 2.6.2 2.6.3 2.6.4 2.6.5 2.7.1 2.7.2 2.7.3 or 3.0.0-alpha4. {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  0m 
34s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
12s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
37s{color} | {color:green} hbase-examples in the patch passed. {color} |
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green}  0m 
 7s{color} | {color:green} The patch does not generate ASF License warnings. 
{color} |
| {color:black}{color} | {color:black} {color} | {color:black} 39m 58s{color} | 
{color:black} {color} |
\\
\\
|| Subsystem || Report/Notes ||
| Docker | Client=1.12.3 Server=1.12.3 Image:yetus/hbase:bdc94b1 |
| JIRA Issue | HBASE-18387 |
| JIRA Patch URL | 
https://issues.apache.org/jira/secure/attachment/12880619/HBASE-18387.master.002.patch
 |
| Optional Tests |  asflicense  javac  javadoc  unit  findbugs  hadoopcheck  
hbaseanti  checkstyle  compile  |
| uname | Linux 880c4eb3bb0a 3.13.0-119-generic #166-Ubuntu SMP Wed May 3 
12:18:55 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | 
/home/jenkins/jenkins-slave/workspace/PreCommit-HBASE-Build/component/dev-support/hbase-personality.sh
 |
| git revision | master / fd76eb3 |
| Default Java | 1.8.0_131 |
| findbugs | v3.1.0-RC3 |
|  Test Results | 
https://builds.apache.org/job/PreCommit-HBASE-Build/7959/testReport/ |
| modules | C: hbase-examples U: hbase-examples |
| Console output | 
https://builds.apache.org/job/PreCommit-HBASE-Build/7959/console |
| Powered by | Apache Yetus 0.4.0   http://yetus.apache.org |


This message was automatically generated.



> [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
>

[jira] [Commented] (HBASE-18387) [Thrift] Make principal configurable in DemoClient.java

2017-08-04 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-18387?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16114495#comment-16114495
 ] 

Hadoop QA commented on HBASE-18387:
---

| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue}  1m 
31s{color} | {color:blue} Docker mode activated. {color} |
| {color:green}+1{color} | {color:green} hbaseanti {color} | {color:green}  0m  
0s{color} | {color:green} Patch does not have any anti-patterns. {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green}  0m  
0s{color} | {color:green} The patch does not contain any @author tags. {color} |
| {color:red}-1{color} | {color:red} test4tests {color} | {color:red}  0m  
0s{color} | {color:red} The patch doesn't appear to include any new or modified 
tests. Please justify why no new tests are needed for this patch. Also please 
list what manual steps were performed to verify this patch. {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  3m 
19s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  0m 
14s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
12s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  0m 
14s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  0m 
26s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
12s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  0m 
15s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  0m 
14s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  0m 
14s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
11s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  0m 
14s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green}  0m 
 0s{color} | {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} hadoopcheck {color} | {color:green} 
30m 15s{color} | {color:green} Patch does not cause any errors with Hadoop 
2.6.1 2.6.2 2.6.3 2.6.4 2.6.5 2.7.1 2.7.2 2.7.3 or 3.0.0-alpha4. {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  0m 
37s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
13s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
37s{color} | {color:green} hbase-examples in the patch passed. {color} |
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green}  0m 
 7s{color} | {color:green} The patch does not generate ASF License warnings. 
{color} |
| {color:black}{color} | {color:black} {color} | {color:black} 39m  9s{color} | 
{color:black} {color} |
\\
\\
|| Subsystem || Report/Notes ||
| Docker | Client=1.12.3 Server=1.12.3 Image:yetus/hbase:bdc94b1 |
| JIRA Issue | HBASE-18387 |
| JIRA Patch URL | 
https://issues.apache.org/jira/secure/attachment/12880396/HBASE-18387.master.001.patch
 |
| Optional Tests |  asflicense  javac  javadoc  unit  findbugs  hadoopcheck  
hbaseanti  checkstyle  compile  |
| uname | Linux 46224b3fb185 3.13.0-119-generic #166-Ubuntu SMP Wed May 3 
12:18:55 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | 
/home/jenkins/jenkins-slave/workspace/PreCommit-HBASE-Build/component/dev-support/hbase-personality.sh
 |
| git revision | master / 6266bb3 |
| Default Java | 1.8.0_131 |
| findbugs | v3.1.0-RC3 |
|  Test Results | 
https://builds.apache.org/job/PreCommit-HBASE-Build/7924/testReport/ |
| modules | C: hbase-examples U: hbase-examples |
| Console output | 
https://builds.apache.org/job/PreCommit-HBASE-Build/7924/console |
| Powered by | Apache Yetus 0.4.0   http://yetus.apache.org |


This message was automatically generated.



> [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
>

[jira] [Commented] (HBASE-18387) [Thrift] Make principal configurable in DemoClient.java

2017-08-04 Thread Sean Busbey (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-18387?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16114467#comment-16114467
 ] 

Sean Busbey commented on HBASE-18387:
-

{code}
-System.out.println("Usage: DemoClient host port [secure=false]");
+System.out.println("Usage: DemoClient host port [secure=false] 
[user=hbase]");
{code} 

Please update the instructions in {{hbase-examples/README.txt}} to explain this 
further. We should specify that it's the principal of the server, and not for 
example a user for the client. Also that it's only needed when talking to a 
secure cluster. Thinking through it, I think that means "server-principal" 
would be better in the CLI help than "user".





> [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
>
>
> 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)


[jira] [Commented] (HBASE-18387) [Thrift] Make principal configurable in DemoClient.java

2017-08-04 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-18387?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16114399#comment-16114399
 ] 

Hadoop QA commented on HBASE-18387:
---

| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue}  0m 
17s{color} | {color:blue} Docker mode activated. {color} |
| {color:green}+1{color} | {color:green} hbaseanti {color} | {color:green}  0m  
0s{color} | {color:green} Patch does not have any anti-patterns. {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green}  0m  
0s{color} | {color:green} The patch does not contain any @author tags. {color} |
| {color:red}-1{color} | {color:red} test4tests {color} | {color:red}  0m  
0s{color} | {color:red} The patch doesn't appear to include any new or modified 
tests. Please justify why no new tests are needed for this patch. Also please 
list what manual steps were performed to verify this patch. {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  3m 
23s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  0m 
14s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
12s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  0m 
14s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  0m 
26s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
12s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  0m 
16s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  0m 
14s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  0m 
14s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
12s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  0m 
14s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green}  0m 
 0s{color} | {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} hadoopcheck {color} | {color:green} 
30m 10s{color} | {color:green} Patch does not cause any errors with Hadoop 
2.6.1 2.6.2 2.6.3 2.6.4 2.6.5 2.7.1 2.7.2 2.7.3 or 3.0.0-alpha4. {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  0m 
33s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
12s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
37s{color} | {color:green} hbase-examples in the patch passed. {color} |
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green}  0m 
 7s{color} | {color:green} The patch does not generate ASF License warnings. 
{color} |
| {color:black}{color} | {color:black} {color} | {color:black} 37m 52s{color} | 
{color:black} {color} |
\\
\\
|| Subsystem || Report/Notes ||
| Docker | Client=1.12.3 Server=1.12.3 Image:yetus/hbase:bdc94b1 |
| JIRA Issue | HBASE-18387 |
| JIRA Patch URL | 
https://issues.apache.org/jira/secure/attachment/12880396/HBASE-18387.master.001.patch
 |
| Optional Tests |  asflicense  javac  javadoc  unit  findbugs  hadoopcheck  
hbaseanti  checkstyle  compile  |
| uname | Linux 4fc8e1048c18 3.13.0-119-generic #166-Ubuntu SMP Wed May 3 
12:18:55 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | 
/home/jenkins/jenkins-slave/workspace/PreCommit-HBASE-Build/component/dev-support/hbase-personality.sh
 |
| git revision | master / 6266bb3 |
| Default Java | 1.8.0_131 |
| findbugs | v3.1.0-RC3 |
|  Test Results | 
https://builds.apache.org/job/PreCommit-HBASE-Build/7916/testReport/ |
| modules | C: hbase-examples U: hbase-examples |
| Console output | 
https://builds.apache.org/job/PreCommit-HBASE-Build/7916/console |
| Powered by | Apache Yetus 0.4.0   http://yetus.apache.org |


This message was automatically generated.



> [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
>