[jira] [Commented] (HBASE-20047) AuthenticationTokenIdentifier should provide a toString

2018-12-14 Thread Hudson (JIRA)


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

Hudson commented on HBASE-20047:


Results for branch branch-1.3
[build #576 on 
builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/branch-1.3/576/]: 
(x) *{color:red}-1 overall{color}*

details (if available):

(/) {color:green}+1 general checks{color}
-- For more information [see general 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-1.3/576//General_Nightly_Build_Report/]


(/) {color:green}+1 jdk7 checks{color}
-- For more information [see jdk7 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-1.3/576//JDK7_Nightly_Build_Report/]


(x) {color:red}-1 jdk8 hadoop2 checks{color}
-- For more information [see jdk8 (hadoop2) 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-1.3/576//JDK8_Nightly_Build_Report_(Hadoop2)/]




(/) {color:green}+1 source release artifact{color}
-- See build output for details.


> AuthenticationTokenIdentifier should provide a toString
> ---
>
> Key: HBASE-20047
> URL: https://issues.apache.org/jira/browse/HBASE-20047
> Project: HBase
>  Issue Type: Improvement
>  Components: Usability
>Reporter: Sean Busbey
>Assignee: maoling
>Priority: Minor
>  Labels: beginner
> Fix For: 3.0.0, 2.1.0, 1.5.0, 1.3.3, 1.4.10
>
> Attachments: HBASE-20047.master.v0.patch, HBASE-20047.master.v1.patch
>
>
> It'd be easier to debug things like MapReduce and Spark jobs if our 
> AuthenticationTokenIdentifier provided a toString method.
> For comparison, here's an example of a MapReduce job that has both an HDFS 
> delegation token and our delegation token:
> {code:java}
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HBASE_AUTH_TOKEN, 
> Service: 92a63bd8-9e00-4c04-ab61-da8e606068e1, Ident: 
> (org.apache.hadoop.hbase.security.token.AuthenticationTokenIdentifier@17)
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HDFS_DELEGATION_TOKEN, 
> Service: 172.31.118.118:8020, Ident: (token for some_user: 
> HDFS_DELEGATION_TOKEN owner=some_u...@example.com, renewer=yarn, realUser=, 
> issueDate=1519274405003, maxDate=1519879205003, sequenceNumber=23, 
> masterKeyId=9)
> {code}
> Stuff in TokenIdentifier is supposed to be public, so we should be fine to 
> dump everything, similar to Hadoop's AbstractDelegationTokenIdentifier.
>  
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (HBASE-20047) AuthenticationTokenIdentifier should provide a toString

2018-12-13 Thread Hudson (JIRA)


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

Hudson commented on HBASE-20047:


SUCCESS: Integrated in Jenkins build HBase-1.3-IT #509 (See 
[https://builds.apache.org/job/HBase-1.3-IT/509/])
HBASE-20047 AuthenticationTokenIdentifier should provide a toString (apurtell: 
rev 35e94c98785095f83c7dd0c7b97b04678fb9c875)
* (edit) 
hbase-client/src/main/java/org/apache/hadoop/hbase/security/token/AuthenticationTokenIdentifier.java


> AuthenticationTokenIdentifier should provide a toString
> ---
>
> Key: HBASE-20047
> URL: https://issues.apache.org/jira/browse/HBASE-20047
> Project: HBase
>  Issue Type: Improvement
>  Components: Usability
>Reporter: Sean Busbey
>Assignee: maoling
>Priority: Minor
>  Labels: beginner
> Fix For: 3.0.0, 2.1.0, 1.5.0, 1.3.3, 1.4.10
>
> Attachments: HBASE-20047.master.v0.patch, HBASE-20047.master.v1.patch
>
>
> It'd be easier to debug things like MapReduce and Spark jobs if our 
> AuthenticationTokenIdentifier provided a toString method.
> For comparison, here's an example of a MapReduce job that has both an HDFS 
> delegation token and our delegation token:
> {code:java}
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HBASE_AUTH_TOKEN, 
> Service: 92a63bd8-9e00-4c04-ab61-da8e606068e1, Ident: 
> (org.apache.hadoop.hbase.security.token.AuthenticationTokenIdentifier@17)
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HDFS_DELEGATION_TOKEN, 
> Service: 172.31.118.118:8020, Ident: (token for some_user: 
> HDFS_DELEGATION_TOKEN owner=some_u...@example.com, renewer=yarn, realUser=, 
> issueDate=1519274405003, maxDate=1519879205003, sequenceNumber=23, 
> masterKeyId=9)
> {code}
> Stuff in TokenIdentifier is supposed to be public, so we should be fine to 
> dump everything, similar to Hadoop's AbstractDelegationTokenIdentifier.
>  
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (HBASE-20047) AuthenticationTokenIdentifier should provide a toString

2018-12-13 Thread Hudson (JIRA)


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

Hudson commented on HBASE-20047:


Results for branch branch-1.4
[build #586 on 
builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/branch-1.4/586/]: 
(x) *{color:red}-1 overall{color}*

details (if available):

(x) {color:red}-1 general checks{color}
-- For more information [see general 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-1.4/586//General_Nightly_Build_Report/]


(x) {color:red}-1 jdk7 checks{color}
-- For more information [see jdk7 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-1.4/586//JDK7_Nightly_Build_Report/]


(x) {color:red}-1 jdk8 hadoop2 checks{color}
-- For more information [see jdk8 (hadoop2) 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-1.4/586//JDK8_Nightly_Build_Report_(Hadoop2)/]




(/) {color:green}+1 source release artifact{color}
-- See build output for details.


> AuthenticationTokenIdentifier should provide a toString
> ---
>
> Key: HBASE-20047
> URL: https://issues.apache.org/jira/browse/HBASE-20047
> Project: HBase
>  Issue Type: Improvement
>  Components: Usability
>Reporter: Sean Busbey
>Assignee: maoling
>Priority: Minor
>  Labels: beginner
> Fix For: 3.0.0, 2.1.0, 1.5.0, 1.3.3, 1.4.10
>
> Attachments: HBASE-20047.master.v0.patch, HBASE-20047.master.v1.patch
>
>
> It'd be easier to debug things like MapReduce and Spark jobs if our 
> AuthenticationTokenIdentifier provided a toString method.
> For comparison, here's an example of a MapReduce job that has both an HDFS 
> delegation token and our delegation token:
> {code:java}
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HBASE_AUTH_TOKEN, 
> Service: 92a63bd8-9e00-4c04-ab61-da8e606068e1, Ident: 
> (org.apache.hadoop.hbase.security.token.AuthenticationTokenIdentifier@17)
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HDFS_DELEGATION_TOKEN, 
> Service: 172.31.118.118:8020, Ident: (token for some_user: 
> HDFS_DELEGATION_TOKEN owner=some_u...@example.com, renewer=yarn, realUser=, 
> issueDate=1519274405003, maxDate=1519879205003, sequenceNumber=23, 
> masterKeyId=9)
> {code}
> Stuff in TokenIdentifier is supposed to be public, so we should be fine to 
> dump everything, similar to Hadoop's AbstractDelegationTokenIdentifier.
>  
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (HBASE-20047) AuthenticationTokenIdentifier should provide a toString

2018-03-27 Thread Hudson (JIRA)

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

Hudson commented on HBASE-20047:


Results for branch HBASE-19064
[build #77 on 
builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/HBASE-19064/77/]: 
(x) *{color:red}-1 overall{color}*

details (if available):

(x) {color:red}-1 general checks{color}
-- For more information [see general 
report|https://builds.apache.org/job/HBase%20Nightly/job/HBASE-19064/77//General_Nightly_Build_Report/]




(x) {color:red}-1 jdk8 hadoop2 checks{color}
-- For more information [see jdk8 (hadoop2) 
report|https://builds.apache.org/job/HBase%20Nightly/job/HBASE-19064/77//JDK8_Nightly_Build_Report_(Hadoop2)/]


(x) {color:red}-1 jdk8 hadoop3 checks{color}
-- For more information [see jdk8 (hadoop3) 
report|https://builds.apache.org/job/HBase%20Nightly/job/HBASE-19064/77//JDK8_Nightly_Build_Report_(Hadoop3)/]


(/) {color:green}+1 source release artifact{color}
-- See build output for details.


> AuthenticationTokenIdentifier should provide a toString
> ---
>
> Key: HBASE-20047
> URL: https://issues.apache.org/jira/browse/HBASE-20047
> Project: HBase
>  Issue Type: Improvement
>  Components: Usability
>Reporter: Sean Busbey
>Assignee: maoling
>Priority: Minor
>  Labels: beginner
> Fix For: 3.0.0, 2.1.0, 1.5.0
>
> Attachments: HBASE-20047.master.v0.patch, HBASE-20047.master.v1.patch
>
>
> It'd be easier to debug things like MapReduce and Spark jobs if our 
> AuthenticationTokenIdentifier provided a toString method.
> For comparison, here's an example of a MapReduce job that has both an HDFS 
> delegation token and our delegation token:
> {code:java}
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HBASE_AUTH_TOKEN, 
> Service: 92a63bd8-9e00-4c04-ab61-da8e606068e1, Ident: 
> (org.apache.hadoop.hbase.security.token.AuthenticationTokenIdentifier@17)
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HDFS_DELEGATION_TOKEN, 
> Service: 172.31.118.118:8020, Ident: (token for some_user: 
> HDFS_DELEGATION_TOKEN owner=some_u...@example.com, renewer=yarn, realUser=, 
> issueDate=1519274405003, maxDate=1519879205003, sequenceNumber=23, 
> masterKeyId=9)
> {code}
> Stuff in TokenIdentifier is supposed to be public, so we should be fine to 
> dump everything, similar to Hadoop's AbstractDelegationTokenIdentifier.
>  
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (HBASE-20047) AuthenticationTokenIdentifier should provide a toString

2018-03-23 Thread Hudson (JIRA)

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

Hudson commented on HBASE-20047:


Results for branch branch-1
[build #258 on 
builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/branch-1/258/]: 
(x) *{color:red}-1 overall{color}*

details (if available):

(x) {color:red}-1 general checks{color}
-- For more information [see general 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-1/258//General_Nightly_Build_Report/]


(x) {color:red}-1 jdk7 checks{color}
-- For more information [see jdk7 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-1/258//JDK7_Nightly_Build_Report/]


(x) {color:red}-1 jdk8 hadoop2 checks{color}
-- For more information [see jdk8 (hadoop2) 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-1/258//JDK8_Nightly_Build_Report_(Hadoop2)/]




(x) {color:red}-1 source release artifact{color}
-- See build output for details.


> AuthenticationTokenIdentifier should provide a toString
> ---
>
> Key: HBASE-20047
> URL: https://issues.apache.org/jira/browse/HBASE-20047
> Project: HBase
>  Issue Type: Improvement
>  Components: Usability
>Reporter: Sean Busbey
>Assignee: maoling
>Priority: Minor
>  Labels: beginner
> Fix For: 3.0.0, 2.1.0, 1.5.0
>
> Attachments: HBASE-20047.master.v0.patch, HBASE-20047.master.v1.patch
>
>
> It'd be easier to debug things like MapReduce and Spark jobs if our 
> AuthenticationTokenIdentifier provided a toString method.
> For comparison, here's an example of a MapReduce job that has both an HDFS 
> delegation token and our delegation token:
> {code:java}
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HBASE_AUTH_TOKEN, 
> Service: 92a63bd8-9e00-4c04-ab61-da8e606068e1, Ident: 
> (org.apache.hadoop.hbase.security.token.AuthenticationTokenIdentifier@17)
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HDFS_DELEGATION_TOKEN, 
> Service: 172.31.118.118:8020, Ident: (token for some_user: 
> HDFS_DELEGATION_TOKEN owner=some_u...@example.com, renewer=yarn, realUser=, 
> issueDate=1519274405003, maxDate=1519879205003, sequenceNumber=23, 
> masterKeyId=9)
> {code}
> Stuff in TokenIdentifier is supposed to be public, so we should be fine to 
> dump everything, similar to Hadoop's AbstractDelegationTokenIdentifier.
>  
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (HBASE-20047) AuthenticationTokenIdentifier should provide a toString

2018-03-21 Thread Andrew Purtell (JIRA)

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

Andrew Purtell commented on HBASE-20047:


No opposition

> AuthenticationTokenIdentifier should provide a toString
> ---
>
> Key: HBASE-20047
> URL: https://issues.apache.org/jira/browse/HBASE-20047
> Project: HBase
>  Issue Type: Improvement
>  Components: Usability
>Reporter: Sean Busbey
>Assignee: maoling
>Priority: Minor
>  Labels: beginner
> Fix For: 3.0.0, 2.1.0
>
> Attachments: HBASE-20047.master.v0.patch, HBASE-20047.master.v1.patch
>
>
> It'd be easier to debug things like MapReduce and Spark jobs if our 
> AuthenticationTokenIdentifier provided a toString method.
> For comparison, here's an example of a MapReduce job that has both an HDFS 
> delegation token and our delegation token:
> {code:java}
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HBASE_AUTH_TOKEN, 
> Service: 92a63bd8-9e00-4c04-ab61-da8e606068e1, Ident: 
> (org.apache.hadoop.hbase.security.token.AuthenticationTokenIdentifier@17)
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HDFS_DELEGATION_TOKEN, 
> Service: 172.31.118.118:8020, Ident: (token for some_user: 
> HDFS_DELEGATION_TOKEN owner=some_u...@example.com, renewer=yarn, realUser=, 
> issueDate=1519274405003, maxDate=1519879205003, sequenceNumber=23, 
> masterKeyId=9)
> {code}
> Stuff in TokenIdentifier is supposed to be public, so we should be fine to 
> dump everything, similar to Hadoop's AbstractDelegationTokenIdentifier.
>  
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (HBASE-20047) AuthenticationTokenIdentifier should provide a toString

2018-03-21 Thread Sean Busbey (JIRA)

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

Sean Busbey commented on HBASE-20047:
-

Any opposition to bringing this back to branch-1? [~apurtell]?

> AuthenticationTokenIdentifier should provide a toString
> ---
>
> Key: HBASE-20047
> URL: https://issues.apache.org/jira/browse/HBASE-20047
> Project: HBase
>  Issue Type: Improvement
>  Components: Usability
>Reporter: Sean Busbey
>Assignee: maoling
>Priority: Minor
>  Labels: beginner
> Fix For: 3.0.0, 2.1.0
>
> Attachments: HBASE-20047.master.v0.patch, HBASE-20047.master.v1.patch
>
>
> It'd be easier to debug things like MapReduce and Spark jobs if our 
> AuthenticationTokenIdentifier provided a toString method.
> For comparison, here's an example of a MapReduce job that has both an HDFS 
> delegation token and our delegation token:
> {code:java}
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HBASE_AUTH_TOKEN, 
> Service: 92a63bd8-9e00-4c04-ab61-da8e606068e1, Ident: 
> (org.apache.hadoop.hbase.security.token.AuthenticationTokenIdentifier@17)
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HDFS_DELEGATION_TOKEN, 
> Service: 172.31.118.118:8020, Ident: (token for some_user: 
> HDFS_DELEGATION_TOKEN owner=some_u...@example.com, renewer=yarn, realUser=, 
> issueDate=1519274405003, maxDate=1519879205003, sequenceNumber=23, 
> masterKeyId=9)
> {code}
> Stuff in TokenIdentifier is supposed to be public, so we should be fine to 
> dump everything, similar to Hadoop's AbstractDelegationTokenIdentifier.
>  
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (HBASE-20047) AuthenticationTokenIdentifier should provide a toString

2018-03-21 Thread Sean Busbey (JIRA)

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

Sean Busbey commented on HBASE-20047:
-

thanks for doing this! sorry I missed the review cycle.

> AuthenticationTokenIdentifier should provide a toString
> ---
>
> Key: HBASE-20047
> URL: https://issues.apache.org/jira/browse/HBASE-20047
> Project: HBase
>  Issue Type: Improvement
>  Components: Usability
>Reporter: Sean Busbey
>Assignee: maoling
>Priority: Minor
>  Labels: beginner
> Fix For: 3.0.0, 2.1.0
>
> Attachments: HBASE-20047.master.v0.patch, HBASE-20047.master.v1.patch
>
>
> It'd be easier to debug things like MapReduce and Spark jobs if our 
> AuthenticationTokenIdentifier provided a toString method.
> For comparison, here's an example of a MapReduce job that has both an HDFS 
> delegation token and our delegation token:
> {code:java}
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HBASE_AUTH_TOKEN, 
> Service: 92a63bd8-9e00-4c04-ab61-da8e606068e1, Ident: 
> (org.apache.hadoop.hbase.security.token.AuthenticationTokenIdentifier@17)
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HDFS_DELEGATION_TOKEN, 
> Service: 172.31.118.118:8020, Ident: (token for some_user: 
> HDFS_DELEGATION_TOKEN owner=some_u...@example.com, renewer=yarn, realUser=, 
> issueDate=1519274405003, maxDate=1519879205003, sequenceNumber=23, 
> masterKeyId=9)
> {code}
> Stuff in TokenIdentifier is supposed to be public, so we should be fine to 
> dump everything, similar to Hadoop's AbstractDelegationTokenIdentifier.
>  
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (HBASE-20047) AuthenticationTokenIdentifier should provide a toString

2018-03-13 Thread Hudson (JIRA)

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

Hudson commented on HBASE-20047:


Results for branch master
[build #261 on 
builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/master/261/]: (x) 
*{color:red}-1 overall{color}*

details (if available):

(/) {color:green}+1 general checks{color}
-- For more information [see general 
report|https://builds.apache.org/job/HBase%20Nightly/job/master/261//General_Nightly_Build_Report/]




(x) {color:red}-1 jdk8 hadoop2 checks{color}
-- For more information [see jdk8 (hadoop2) 
report|https://builds.apache.org/job/HBase%20Nightly/job/master/261//JDK8_Nightly_Build_Report_(Hadoop2)/]


(x) {color:red}-1 jdk8 hadoop3 checks{color}
-- For more information [see jdk8 (hadoop3) 
report|https://builds.apache.org/job/HBase%20Nightly/job/master/261//JDK8_Nightly_Build_Report_(Hadoop3)/]


(/) {color:green}+1 source release artifact{color}
-- See build output for details.


> AuthenticationTokenIdentifier should provide a toString
> ---
>
> Key: HBASE-20047
> URL: https://issues.apache.org/jira/browse/HBASE-20047
> Project: HBase
>  Issue Type: Improvement
>  Components: Usability
>Reporter: Sean Busbey
>Assignee: maoling
>Priority: Minor
>  Labels: beginner
> Fix For: 3.0.0, 2.1.0
>
> Attachments: HBASE-20047.master.v0.patch, HBASE-20047.master.v1.patch
>
>
> It'd be easier to debug things like MapReduce and Spark jobs if our 
> AuthenticationTokenIdentifier provided a toString method.
> For comparison, here's an example of a MapReduce job that has both an HDFS 
> delegation token and our delegation token:
> {code:java}
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HBASE_AUTH_TOKEN, 
> Service: 92a63bd8-9e00-4c04-ab61-da8e606068e1, Ident: 
> (org.apache.hadoop.hbase.security.token.AuthenticationTokenIdentifier@17)
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HDFS_DELEGATION_TOKEN, 
> Service: 172.31.118.118:8020, Ident: (token for some_user: 
> HDFS_DELEGATION_TOKEN owner=some_u...@example.com, renewer=yarn, realUser=, 
> issueDate=1519274405003, maxDate=1519879205003, sequenceNumber=23, 
> masterKeyId=9)
> {code}
> Stuff in TokenIdentifier is supposed to be public, so we should be fine to 
> dump everything, similar to Hadoop's AbstractDelegationTokenIdentifier.
>  
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (HBASE-20047) AuthenticationTokenIdentifier should provide a toString

2018-03-12 Thread Hudson (JIRA)

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

Hudson commented on HBASE-20047:


Results for branch branch-2
[build #478 on 
builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/478/]: 
(x) *{color:red}-1 overall{color}*

details (if available):

(/) {color:green}+1 general checks{color}
-- For more information [see general 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/478//General_Nightly_Build_Report/]




(/) {color:green}+1 jdk8 hadoop2 checks{color}
-- For more information [see jdk8 (hadoop2) 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/478//JDK8_Nightly_Build_Report_(Hadoop2)/]


(/) {color:green}+1 jdk8 hadoop3 checks{color}
-- For more information [see jdk8 (hadoop3) 
report|https://builds.apache.org/job/HBase%20Nightly/job/branch-2/478//JDK8_Nightly_Build_Report_(Hadoop3)/]


(/) {color:green}+1 source release artifact{color}
-- See build output for details.


> AuthenticationTokenIdentifier should provide a toString
> ---
>
> Key: HBASE-20047
> URL: https://issues.apache.org/jira/browse/HBASE-20047
> Project: HBase
>  Issue Type: Improvement
>  Components: Usability
>Reporter: Sean Busbey
>Assignee: maoling
>Priority: Minor
>  Labels: beginner
> Fix For: 3.0.0, 2.1.0
>
> Attachments: HBASE-20047.master.v0.patch, HBASE-20047.master.v1.patch
>
>
> It'd be easier to debug things like MapReduce and Spark jobs if our 
> AuthenticationTokenIdentifier provided a toString method.
> For comparison, here's an example of a MapReduce job that has both an HDFS 
> delegation token and our delegation token:
> {code:java}
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HBASE_AUTH_TOKEN, 
> Service: 92a63bd8-9e00-4c04-ab61-da8e606068e1, Ident: 
> (org.apache.hadoop.hbase.security.token.AuthenticationTokenIdentifier@17)
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HDFS_DELEGATION_TOKEN, 
> Service: 172.31.118.118:8020, Ident: (token for some_user: 
> HDFS_DELEGATION_TOKEN owner=some_u...@example.com, renewer=yarn, realUser=, 
> issueDate=1519274405003, maxDate=1519879205003, sequenceNumber=23, 
> masterKeyId=9)
> {code}
> Stuff in TokenIdentifier is supposed to be public, so we should be fine to 
> dump everything, similar to Hadoop's AbstractDelegationTokenIdentifier.
>  
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (HBASE-20047) AuthenticationTokenIdentifier should provide a toString

2018-03-12 Thread maoling (JIRA)

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

maoling commented on HBASE-20047:
-

YES!!

> AuthenticationTokenIdentifier should provide a toString
> ---
>
> Key: HBASE-20047
> URL: https://issues.apache.org/jira/browse/HBASE-20047
> Project: HBase
>  Issue Type: Improvement
>  Components: Usability
>Reporter: Sean Busbey
>Assignee: maoling
>Priority: Minor
>  Labels: beginner
> Fix For: 3.0.0, 2.1.0
>
> Attachments: HBASE-20047.master.v0.patch, HBASE-20047.master.v1.patch
>
>
> It'd be easier to debug things like MapReduce and Spark jobs if our 
> AuthenticationTokenIdentifier provided a toString method.
> For comparison, here's an example of a MapReduce job that has both an HDFS 
> delegation token and our delegation token:
> {code:java}
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HBASE_AUTH_TOKEN, 
> Service: 92a63bd8-9e00-4c04-ab61-da8e606068e1, Ident: 
> (org.apache.hadoop.hbase.security.token.AuthenticationTokenIdentifier@17)
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HDFS_DELEGATION_TOKEN, 
> Service: 172.31.118.118:8020, Ident: (token for some_user: 
> HDFS_DELEGATION_TOKEN owner=some_u...@example.com, renewer=yarn, realUser=, 
> issueDate=1519274405003, maxDate=1519879205003, sequenceNumber=23, 
> masterKeyId=9)
> {code}
> Stuff in TokenIdentifier is supposed to be public, so we should be fine to 
> dump everything, similar to Hadoop's AbstractDelegationTokenIdentifier.
>  
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (HBASE-20047) AuthenticationTokenIdentifier should provide a toString

2018-03-12 Thread Chia-Ping Tsai (JIRA)

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

Chia-Ping Tsai commented on HBASE-20047:


{quote}my [githubId|https://github.com/maoling]
{quote}
maoling  <-- Is it correct info?

> AuthenticationTokenIdentifier should provide a toString
> ---
>
> Key: HBASE-20047
> URL: https://issues.apache.org/jira/browse/HBASE-20047
> Project: HBase
>  Issue Type: Improvement
>  Components: Usability
>Reporter: Sean Busbey
>Assignee: maoling
>Priority: Minor
>  Labels: beginner
> Fix For: 3.0.0, 2.1.0
>
> Attachments: HBASE-20047.master.v0.patch, HBASE-20047.master.v1.patch
>
>
> It'd be easier to debug things like MapReduce and Spark jobs if our 
> AuthenticationTokenIdentifier provided a toString method.
> For comparison, here's an example of a MapReduce job that has both an HDFS 
> delegation token and our delegation token:
> {code:java}
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HBASE_AUTH_TOKEN, 
> Service: 92a63bd8-9e00-4c04-ab61-da8e606068e1, Ident: 
> (org.apache.hadoop.hbase.security.token.AuthenticationTokenIdentifier@17)
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HDFS_DELEGATION_TOKEN, 
> Service: 172.31.118.118:8020, Ident: (token for some_user: 
> HDFS_DELEGATION_TOKEN owner=some_u...@example.com, renewer=yarn, realUser=, 
> issueDate=1519274405003, maxDate=1519879205003, sequenceNumber=23, 
> masterKeyId=9)
> {code}
> Stuff in TokenIdentifier is supposed to be public, so we should be fine to 
> dump everything, similar to Hadoop's AbstractDelegationTokenIdentifier.
>  
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (HBASE-20047) AuthenticationTokenIdentifier should provide a toString

2018-03-12 Thread Chia-Ping Tsai (JIRA)

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

Chia-Ping Tsai commented on HBASE-20047:


{quote}this is also a very trival patch, would you please commit it together?
{quote}
I will take a look later. Let me commit this patch first.

> AuthenticationTokenIdentifier should provide a toString
> ---
>
> Key: HBASE-20047
> URL: https://issues.apache.org/jira/browse/HBASE-20047
> Project: HBase
>  Issue Type: Improvement
>  Components: Usability
>Reporter: Sean Busbey
>Assignee: maoling
>Priority: Minor
>  Labels: beginner
> Fix For: 3.0.0, 2.1.0
>
> Attachments: HBASE-20047.master.v0.patch, HBASE-20047.master.v1.patch
>
>
> It'd be easier to debug things like MapReduce and Spark jobs if our 
> AuthenticationTokenIdentifier provided a toString method.
> For comparison, here's an example of a MapReduce job that has both an HDFS 
> delegation token and our delegation token:
> {code:java}
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HBASE_AUTH_TOKEN, 
> Service: 92a63bd8-9e00-4c04-ab61-da8e606068e1, Ident: 
> (org.apache.hadoop.hbase.security.token.AuthenticationTokenIdentifier@17)
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HDFS_DELEGATION_TOKEN, 
> Service: 172.31.118.118:8020, Ident: (token for some_user: 
> HDFS_DELEGATION_TOKEN owner=some_u...@example.com, renewer=yarn, realUser=, 
> issueDate=1519274405003, maxDate=1519879205003, sequenceNumber=23, 
> masterKeyId=9)
> {code}
> Stuff in TokenIdentifier is supposed to be public, so we should be fine to 
> dump everything, similar to Hadoop's AbstractDelegationTokenIdentifier.
>  
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (HBASE-20047) AuthenticationTokenIdentifier should provide a toString

2018-03-12 Thread maoling (JIRA)

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

maoling commented on HBASE-20047:
-

Thanks [~chia7712]

my [githubId|https://github.com/maoling]

this is also a very trival patch, would you please commit it together?

> AuthenticationTokenIdentifier should provide a toString
> ---
>
> Key: HBASE-20047
> URL: https://issues.apache.org/jira/browse/HBASE-20047
> Project: HBase
>  Issue Type: Improvement
>  Components: Usability
>Reporter: Sean Busbey
>Assignee: maoling
>Priority: Minor
>  Labels: beginner
> Attachments: HBASE-20047.master.v0.patch, HBASE-20047.master.v1.patch
>
>
> It'd be easier to debug things like MapReduce and Spark jobs if our 
> AuthenticationTokenIdentifier provided a toString method.
> For comparison, here's an example of a MapReduce job that has both an HDFS 
> delegation token and our delegation token:
> {code:java}
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HBASE_AUTH_TOKEN, 
> Service: 92a63bd8-9e00-4c04-ab61-da8e606068e1, Ident: 
> (org.apache.hadoop.hbase.security.token.AuthenticationTokenIdentifier@17)
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HDFS_DELEGATION_TOKEN, 
> Service: 172.31.118.118:8020, Ident: (token for some_user: 
> HDFS_DELEGATION_TOKEN owner=some_u...@example.com, renewer=yarn, realUser=, 
> issueDate=1519274405003, maxDate=1519879205003, sequenceNumber=23, 
> masterKeyId=9)
> {code}
> Stuff in TokenIdentifier is supposed to be public, so we should be fine to 
> dump everything, similar to Hadoop's AbstractDelegationTokenIdentifier.
>  
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (HBASE-20047) AuthenticationTokenIdentifier should provide a toString

2018-03-10 Thread Chia-Ping Tsai (JIRA)

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

Chia-Ping Tsai commented on HBASE-20047:


Will commit it tomorrow if no objections.

> AuthenticationTokenIdentifier should provide a toString
> ---
>
> Key: HBASE-20047
> URL: https://issues.apache.org/jira/browse/HBASE-20047
> Project: HBase
>  Issue Type: Improvement
>  Components: Usability
>Reporter: Sean Busbey
>Assignee: maoling
>Priority: Minor
>  Labels: beginner
> Attachments: HBASE-20047.master.v0.patch, HBASE-20047.master.v1.patch
>
>
> It'd be easier to debug things like MapReduce and Spark jobs if our 
> AuthenticationTokenIdentifier provided a toString method.
> For comparison, here's an example of a MapReduce job that has both an HDFS 
> delegation token and our delegation token:
> {code:java}
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HBASE_AUTH_TOKEN, 
> Service: 92a63bd8-9e00-4c04-ab61-da8e606068e1, Ident: 
> (org.apache.hadoop.hbase.security.token.AuthenticationTokenIdentifier@17)
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HDFS_DELEGATION_TOKEN, 
> Service: 172.31.118.118:8020, Ident: (token for some_user: 
> HDFS_DELEGATION_TOKEN owner=some_u...@example.com, renewer=yarn, realUser=, 
> issueDate=1519274405003, maxDate=1519879205003, sequenceNumber=23, 
> masterKeyId=9)
> {code}
> Stuff in TokenIdentifier is supposed to be public, so we should be fine to 
> dump everything, similar to Hadoop's AbstractDelegationTokenIdentifier.
>  
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (HBASE-20047) AuthenticationTokenIdentifier should provide a toString

2018-03-10 Thread Chia-Ping Tsai (JIRA)

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

Chia-Ping Tsai commented on HBASE-20047:


+1

> AuthenticationTokenIdentifier should provide a toString
> ---
>
> Key: HBASE-20047
> URL: https://issues.apache.org/jira/browse/HBASE-20047
> Project: HBase
>  Issue Type: Improvement
>  Components: Usability
>Reporter: Sean Busbey
>Assignee: maoling
>Priority: Minor
>  Labels: beginner
> Attachments: HBASE-20047.master.v0.patch, HBASE-20047.master.v1.patch
>
>
> It'd be easier to debug things like MapReduce and Spark jobs if our 
> AuthenticationTokenIdentifier provided a toString method.
> For comparison, here's an example of a MapReduce job that has both an HDFS 
> delegation token and our delegation token:
> {code:java}
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HBASE_AUTH_TOKEN, 
> Service: 92a63bd8-9e00-4c04-ab61-da8e606068e1, Ident: 
> (org.apache.hadoop.hbase.security.token.AuthenticationTokenIdentifier@17)
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HDFS_DELEGATION_TOKEN, 
> Service: 172.31.118.118:8020, Ident: (token for some_user: 
> HDFS_DELEGATION_TOKEN owner=some_u...@example.com, renewer=yarn, realUser=, 
> issueDate=1519274405003, maxDate=1519879205003, sequenceNumber=23, 
> masterKeyId=9)
> {code}
> Stuff in TokenIdentifier is supposed to be public, so we should be fine to 
> dump everything, similar to Hadoop's AbstractDelegationTokenIdentifier.
>  
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (HBASE-20047) AuthenticationTokenIdentifier should provide a toString

2018-03-02 Thread maoling (JIRA)

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

maoling commented on HBASE-20047:
-

[~busbey]

> AuthenticationTokenIdentifier should provide a toString
> ---
>
> Key: HBASE-20047
> URL: https://issues.apache.org/jira/browse/HBASE-20047
> Project: HBase
>  Issue Type: Improvement
>  Components: Usability
>Reporter: Sean Busbey
>Assignee: maoling
>Priority: Minor
>  Labels: beginner
> Attachments: HBASE-20047.master.v0.patch, HBASE-20047.master.v1.patch
>
>
> It'd be easier to debug things like MapReduce and Spark jobs if our 
> AuthenticationTokenIdentifier provided a toString method.
> For comparison, here's an example of a MapReduce job that has both an HDFS 
> delegation token and our delegation token:
> {code:java}
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HBASE_AUTH_TOKEN, 
> Service: 92a63bd8-9e00-4c04-ab61-da8e606068e1, Ident: 
> (org.apache.hadoop.hbase.security.token.AuthenticationTokenIdentifier@17)
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HDFS_DELEGATION_TOKEN, 
> Service: 172.31.118.118:8020, Ident: (token for some_user: 
> HDFS_DELEGATION_TOKEN owner=some_u...@example.com, renewer=yarn, realUser=, 
> issueDate=1519274405003, maxDate=1519879205003, sequenceNumber=23, 
> masterKeyId=9)
> {code}
> Stuff in TokenIdentifier is supposed to be public, so we should be fine to 
> dump everything, similar to Hadoop's AbstractDelegationTokenIdentifier.
>  
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (HBASE-20047) AuthenticationTokenIdentifier should provide a toString

2018-02-27 Thread Chia-Ping Tsai (JIRA)

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

Chia-Ping Tsai commented on HBASE-20047:


[~busbey] Is it good to commit?

> AuthenticationTokenIdentifier should provide a toString
> ---
>
> Key: HBASE-20047
> URL: https://issues.apache.org/jira/browse/HBASE-20047
> Project: HBase
>  Issue Type: Improvement
>  Components: Usability
>Reporter: Sean Busbey
>Assignee: maoling
>Priority: Minor
>  Labels: beginner
> Attachments: HBASE-20047.master.v0.patch, HBASE-20047.master.v1.patch
>
>
> It'd be easier to debug things like MapReduce and Spark jobs if our 
> AuthenticationTokenIdentifier provided a toString method.
> For comparison, here's an example of a MapReduce job that has both an HDFS 
> delegation token and our delegation token:
> {code:java}
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HBASE_AUTH_TOKEN, 
> Service: 92a63bd8-9e00-4c04-ab61-da8e606068e1, Ident: 
> (org.apache.hadoop.hbase.security.token.AuthenticationTokenIdentifier@17)
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HDFS_DELEGATION_TOKEN, 
> Service: 172.31.118.118:8020, Ident: (token for some_user: 
> HDFS_DELEGATION_TOKEN owner=some_u...@example.com, renewer=yarn, realUser=, 
> issueDate=1519274405003, maxDate=1519879205003, sequenceNumber=23, 
> masterKeyId=9)
> {code}
> Stuff in TokenIdentifier is supposed to be public, so we should be fine to 
> dump everything, similar to Hadoop's AbstractDelegationTokenIdentifier.
>  
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (HBASE-20047) AuthenticationTokenIdentifier should provide a toString

2018-02-24 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-20047:
---

| (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:brown} Prechecks {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:brown} master Compile Tests {color} ||
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  4m 
19s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  0m 
20s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
28s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} shadedjars {color} | {color:green}  5m 
13s{color} | {color:green} branch has no errors when building our shaded 
downstream artifacts. {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  0m 
50s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
19s{color} | {color:green} master passed {color} |
|| || || || {color:brown} Patch Compile Tests {color} ||
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  4m 
21s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  0m 
20s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  0m 
20s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
29s{color} | {color:green} the patch passed {color} |
| {color:red}-1{color} | {color:red} whitespace {color} | {color:red}  0m  
0s{color} | {color:red} The patch has 1 line(s) that end in whitespace. Use git 
apply --whitespace=fix <>. Refer https://git-scm.com/docs/git-apply 
{color} |
| {color:green}+1{color} | {color:green} shadedjars {color} | {color:green}  4m 
48s{color} | {color:green} patch has no errors when building our shaded 
downstream artifacts. {color} |
| {color:green}+1{color} | {color:green} hadoopcheck {color} | {color:green} 
18m 40s{color} | {color:green} Patch does not cause any errors with Hadoop 
2.6.5 2.7.4 or 3.0.0. {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  0m 
59s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
19s{color} | {color:green} the patch passed {color} |
|| || || || {color:brown} Other Tests {color} ||
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  2m 
55s{color} | {color:green} hbase-client in the patch passed. {color} |
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green}  0m 
 9s{color} | {color:green} The patch does not generate ASF License warnings. 
{color} |
| {color:black}{color} | {color:black} {color} | {color:black} 39m 49s{color} | 
{color:black} {color} |
\\
\\
|| Subsystem || Report/Notes ||
| Docker | Client=17.05.0-ce Server=17.05.0-ce Image:yetus/hbase:eee3b01 |
| JIRA Issue | HBASE-20047 |
| JIRA Patch URL | 
https://issues.apache.org/jira/secure/attachment/12911948/HBASE-20047.master.v1.patch
 |
| Optional Tests |  asflicense  javac  javadoc  unit  findbugs  shadedjars  
hadoopcheck  hbaseanti  checkstyle  compile  |
| uname | Linux a3c1f875ae26 3.13.0-139-generic #188-Ubuntu SMP Tue Jan 9 
14:43:09 UTC 2018 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 / 549a6d93d4 |
| maven | version: Apache Maven 3.5.2 
(138edd61fd100ec658bfa2d307c43b76940a5d7d; 2017-10-18T07:58:13Z) |
| Default Java | 1.8.0_151 |
| findbugs | v3.1.0-RC3 |
| whitespace | 
https://builds.apache.org/job/PreCommit-HBASE-Build/11665/artifact/patchprocess/whitespace-eol.txt
 |
|  Test Results | 
https://builds.apache.org/job/PreCommit-HBASE-Build/11665/testReport/ |
| 

[jira] [Commented] (HBASE-20047) AuthenticationTokenIdentifier should provide a toString

2018-02-24 Thread maoling (JIRA)

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

maoling commented on HBASE-20047:
-

v0 just follows an example of Hadoop's 
[AbstractDelegationTokenIdentifier|https://github.uconn.edu/SNSL/hadoop/blob/master/code/stock_hadoop/hadoop-1.1.2/src/core/org/apache/hadoop/security/token/delegation/AbstractDelegationTokenIdentifier.java#L189]

v1 has used + operator to concat the string
{code:java}
AuthenticationTokenIdentifier ati = new AuthenticationTokenIdentifier();
ati.setUsername("maoling");
ati.setKeyId(123);
ati.setIssueDate(System.currentTimeMillis());
ati.setExpirationDate(System.currentTimeMillis());
ati.setSequenceNumber(System.currentTimeMillis());
System.out.println("the format of the returned string is: \n" + ati);
{code}
the format of the returned string is:
{code:java}
(username=maoling, keyId=123, issueDate=1519538066607, 
expirationDate=1519538066607, sequenceNumber=1519538066607)
{code}

> AuthenticationTokenIdentifier should provide a toString
> ---
>
> Key: HBASE-20047
> URL: https://issues.apache.org/jira/browse/HBASE-20047
> Project: HBase
>  Issue Type: Improvement
>  Components: Usability
>Reporter: Sean Busbey
>Assignee: maoling
>Priority: Minor
>  Labels: beginner
> Attachments: HBASE-20047.master.v0.patch, HBASE-20047.master.v1.patch
>
>
> It'd be easier to debug things like MapReduce and Spark jobs if our 
> AuthenticationTokenIdentifier provided a toString method.
> For comparison, here's an example of a MapReduce job that has both an HDFS 
> delegation token and our delegation token:
> {code:java}
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HBASE_AUTH_TOKEN, 
> Service: 92a63bd8-9e00-4c04-ab61-da8e606068e1, Ident: 
> (org.apache.hadoop.hbase.security.token.AuthenticationTokenIdentifier@17)
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HDFS_DELEGATION_TOKEN, 
> Service: 172.31.118.118:8020, Ident: (token for some_user: 
> HDFS_DELEGATION_TOKEN owner=some_u...@example.com, renewer=yarn, realUser=, 
> issueDate=1519274405003, maxDate=1519879205003, sequenceNumber=23, 
> masterKeyId=9)
> {code}
> Stuff in TokenIdentifier is supposed to be public, so we should be fine to 
> dump everything, similar to Hadoop's AbstractDelegationTokenIdentifier.
>  
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (HBASE-20047) AuthenticationTokenIdentifier should provide a toString

2018-02-24 Thread Sean Busbey (JIRA)

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

Sean Busbey commented on HBASE-20047:
-

{code}
-1  test4tests  0m 0s   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.
{code}

I don't think any tests are needed for a newly added toString method, since I 
don't think we want to make any assertions about the format returned string for 
the future. I would like to see that someone spins up an instance with this 
change in place to show what the updated log message looks like before we 
commit the change.

> AuthenticationTokenIdentifier should provide a toString
> ---
>
> Key: HBASE-20047
> URL: https://issues.apache.org/jira/browse/HBASE-20047
> Project: HBase
>  Issue Type: Improvement
>  Components: Usability
>Reporter: Sean Busbey
>Assignee: maoling
>Priority: Minor
>  Labels: beginner
> Attachments: HBASE-20047.master.v0.patch
>
>
> It'd be easier to debug things like MapReduce and Spark jobs if our 
> AuthenticationTokenIdentifier provided a toString method.
> For comparison, here's an example of a MapReduce job that has both an HDFS 
> delegation token and our delegation token:
> {code:java}
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HBASE_AUTH_TOKEN, 
> Service: 92a63bd8-9e00-4c04-ab61-da8e606068e1, Ident: 
> (org.apache.hadoop.hbase.security.token.AuthenticationTokenIdentifier@17)
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HDFS_DELEGATION_TOKEN, 
> Service: 172.31.118.118:8020, Ident: (token for some_user: 
> HDFS_DELEGATION_TOKEN owner=some_u...@example.com, renewer=yarn, realUser=, 
> issueDate=1519274405003, maxDate=1519879205003, sequenceNumber=23, 
> masterKeyId=9)
> {code}
> Stuff in TokenIdentifier is supposed to be public, so we should be fine to 
> dump everything, similar to Hadoop's AbstractDelegationTokenIdentifier.
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (HBASE-20047) AuthenticationTokenIdentifier should provide a toString

2018-02-24 Thread Sean Busbey (JIRA)

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

Sean Busbey commented on HBASE-20047:
-

agreed. if we're concerned about performance here, make sure all the concats 
are done via StringBuilder.append. If we're not so concerned, then {{javac}} 
ought to turn one big block of + operators into a StringBuilder internally 
anyways, so just use them directly.

> AuthenticationTokenIdentifier should provide a toString
> ---
>
> Key: HBASE-20047
> URL: https://issues.apache.org/jira/browse/HBASE-20047
> Project: HBase
>  Issue Type: Improvement
>  Components: Usability
>Reporter: Sean Busbey
>Priority: Minor
>  Labels: beginner
> Attachments: HBASE-20047.master.v0.patch
>
>
> It'd be easier to debug things like MapReduce and Spark jobs if our 
> AuthenticationTokenIdentifier provided a toString method.
> For comparison, here's an example of a MapReduce job that has both an HDFS 
> delegation token and our delegation token:
> {code:java}
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HBASE_AUTH_TOKEN, 
> Service: 92a63bd8-9e00-4c04-ab61-da8e606068e1, Ident: 
> (org.apache.hadoop.hbase.security.token.AuthenticationTokenIdentifier@17)
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HDFS_DELEGATION_TOKEN, 
> Service: 172.31.118.118:8020, Ident: (token for some_user: 
> HDFS_DELEGATION_TOKEN owner=some_u...@example.com, renewer=yarn, realUser=, 
> issueDate=1519274405003, maxDate=1519879205003, sequenceNumber=23, 
> masterKeyId=9)
> {code}
> Stuff in TokenIdentifier is supposed to be public, so we should be fine to 
> dump everything, similar to Hadoop's AbstractDelegationTokenIdentifier.
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (HBASE-20047) AuthenticationTokenIdentifier should provide a toString

2018-02-24 Thread Chia-Ping Tsai (JIRA)

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

Chia-Ping Tsai commented on HBASE-20047:


We instantiate a StringBuilder but the StringBuilder#append isn't used to 
concat the multiple strings. Perhaps just use + operator to concat the string? 

{code}
+  public String toString() {
+StringBuilder buffer = new StringBuilder();
+buffer
+.append("username=" + username + ", keyId=" + keyId + ", 
issueDate=" + issueDate
+   + ", expirationDate=" + expirationDate + ", 
sequenceNumber=" + sequenceNumber);
+return buffer.toString();
+  }
{code}

> AuthenticationTokenIdentifier should provide a toString
> ---
>
> Key: HBASE-20047
> URL: https://issues.apache.org/jira/browse/HBASE-20047
> Project: HBase
>  Issue Type: Improvement
>  Components: Usability
>Reporter: Sean Busbey
>Priority: Minor
>  Labels: beginner
> Attachments: HBASE-20047.master.v0.patch
>
>
> It'd be easier to debug things like MapReduce and Spark jobs if our 
> AuthenticationTokenIdentifier provided a toString method.
> For comparison, here's an example of a MapReduce job that has both an HDFS 
> delegation token and our delegation token:
> {code:java}
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HBASE_AUTH_TOKEN, 
> Service: 92a63bd8-9e00-4c04-ab61-da8e606068e1, Ident: 
> (org.apache.hadoop.hbase.security.token.AuthenticationTokenIdentifier@17)
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HDFS_DELEGATION_TOKEN, 
> Service: 172.31.118.118:8020, Ident: (token for some_user: 
> HDFS_DELEGATION_TOKEN owner=some_u...@example.com, renewer=yarn, realUser=, 
> issueDate=1519274405003, maxDate=1519879205003, sequenceNumber=23, 
> masterKeyId=9)
> {code}
> Stuff in TokenIdentifier is supposed to be public, so we should be fine to 
> dump everything, similar to Hadoop's AbstractDelegationTokenIdentifier.
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (HBASE-20047) AuthenticationTokenIdentifier should provide a toString

2018-02-24 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-20047:
---

| (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:brown} Prechecks {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:brown} master Compile Tests {color} ||
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  5m 
49s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  0m 
24s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
32s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} shadedjars {color} | {color:green}  6m 
19s{color} | {color:green} branch has no errors when building our shaded 
downstream artifacts. {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  1m 
15s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
25s{color} | {color:green} master passed {color} |
|| || || || {color:brown} Patch Compile Tests {color} ||
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  5m 
31s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  0m 
29s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  0m 
29s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
34s{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} shadedjars {color} | {color:green}  5m 
45s{color} | {color:green} patch has no errors when building our shaded 
downstream artifacts. {color} |
| {color:green}+1{color} | {color:green} hadoopcheck {color} | {color:green} 
22m 51s{color} | {color:green} Patch does not cause any errors with Hadoop 
2.6.5 2.7.4 or 3.0.0. {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  1m 
13s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
23s{color} | {color:green} the patch passed {color} |
|| || || || {color:brown} Other Tests {color} ||
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  3m 
11s{color} | {color:green} hbase-client in the patch passed. {color} |
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green}  0m 
12s{color} | {color:green} The patch does not generate ASF License warnings. 
{color} |
| {color:black}{color} | {color:black} {color} | {color:black} 49m 25s{color} | 
{color:black} {color} |
\\
\\
|| Subsystem || Report/Notes ||
| Docker | Client=17.05.0-ce Server=17.05.0-ce Image:yetus/hbase:eee3b01 |
| JIRA Issue | HBASE-20047 |
| JIRA Patch URL | 
https://issues.apache.org/jira/secure/attachment/12911880/HBASE-20047.master.v0.patch
 |
| Optional Tests |  asflicense  javac  javadoc  unit  findbugs  shadedjars  
hadoopcheck  hbaseanti  checkstyle  compile  |
| uname | Linux ed4712b1995f 3.13.0-137-generic #186-Ubuntu SMP Mon Dec 4 
19:09:19 UTC 2017 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 / bbbfeae25e |
| maven | version: Apache Maven 3.5.2 
(138edd61fd100ec658bfa2d307c43b76940a5d7d; 2017-10-18T07:58:13Z) |
| Default Java | 1.8.0_151 |
| findbugs | v3.1.0-RC3 |
|  Test Results | 
https://builds.apache.org/job/PreCommit-HBASE-Build/11655/testReport/ |
| Max. process+thread count | 277 (vs. ulimit of 1) |
| modules | C: hbase-client U: hbase-client |
| Console output | 
https://builds.apache.org/job/PreCommit-HBASE-Build/11655/console |
| Powered 

[jira] [Commented] (HBASE-20047) AuthenticationTokenIdentifier should provide a toString

2018-02-24 Thread Reid Chan (JIRA)

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

Reid Chan commented on HBASE-20047:
---

LGTM

> AuthenticationTokenIdentifier should provide a toString
> ---
>
> Key: HBASE-20047
> URL: https://issues.apache.org/jira/browse/HBASE-20047
> Project: HBase
>  Issue Type: Improvement
>  Components: Usability
>Reporter: Sean Busbey
>Priority: Minor
>  Labels: beginner
> Attachments: HBASE-20047.master.v0.patch
>
>
> It'd be easier to debug things like MapReduce and Spark jobs if our 
> AuthenticationTokenIdentifier provided a toString method.
> For comparison, here's an example of a MapReduce job that has both an HDFS 
> delegation token and our delegation token:
> {code:java}
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HBASE_AUTH_TOKEN, 
> Service: 92a63bd8-9e00-4c04-ab61-da8e606068e1, Ident: 
> (org.apache.hadoop.hbase.security.token.AuthenticationTokenIdentifier@17)
> 18/02/21 20:40:06 INFO mapreduce.JobSubmitter: Kind: HDFS_DELEGATION_TOKEN, 
> Service: 172.31.118.118:8020, Ident: (token for some_user: 
> HDFS_DELEGATION_TOKEN owner=some_u...@example.com, renewer=yarn, realUser=, 
> issueDate=1519274405003, maxDate=1519879205003, sequenceNumber=23, 
> masterKeyId=9)
> {code}
> Stuff in TokenIdentifier is supposed to be public, so we should be fine to 
> dump everything, similar to Hadoop's AbstractDelegationTokenIdentifier.
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)