[
https://issues.apache.org/jira/browse/HDFS-10211?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15211123#comment-15211123
]
Chris Nauroth commented on HDFS-10211:
--------------------------------------
Hi [~yzhangal]. Is this a duplicate of HDFS-9732?
bq. I hope no code really depends on the output string.
This {{toString}} output is used for the output of the {{hdfs fetchdt}] shell
command, and therefore it has to be covered by our backwards-compatibility
policy. HDFS-9732 has more discussion of this and a few ideas for how to
proceed.
> Add more info to DelegationTokenIdentifier#toString for better supportability
> -----------------------------------------------------------------------------
>
> Key: HDFS-10211
> URL: https://issues.apache.org/jira/browse/HDFS-10211
> Project: Hadoop HDFS
> Issue Type: Improvement
> Reporter: Yongjun Zhang
> Assignee: Yongjun Zhang
>
> Base class {{AbstractDelegationTokenIdentifier}} has the following
> implementation of {{toString()}} method
> {code}
> @Override
> public String toString() {
> StringBuilder buffer = new StringBuilder();
> buffer
> .append("owner=" + owner + ", renewer=" + renewer + ", realUser="
> + realUser + ", issueDate=" + issueDate + ", maxDate=" + maxDate
> + ", sequenceNumber=" + sequenceNumber + ", masterKeyId="
> + masterKeyId);
> return buffer.toString();
> }
> {code}
> However, derived class
> {{org.apache.hadoop.hdfs.security.token.delegation.DelegationTokenIdentifier}}
> has the following implementation that overrides the base class above:
> {code}
> @Override
> public String toString() {
> return getKind() + " token " + getSequenceNumber()
> + " for " + getUser().getShortUserName();
> }
> {code}
> And when exception is thrown because of token expiration or other reason:
> {code}
> if (info.getRenewDate() < Time.now()) {
> throw new InvalidToken("token (" + identifier.toString() + ") is
> expired");
> }
> {code}
> The exception doesn't show the detailed information about the token, like the
> base class' toString() method returns.
> Creating this jira to change the
> {{org.apache.hadoop.hdfs.security.token.delegation.DelegationTokenIdentifier}}
> implementation to include all the info about the token, as included by the
> base class.
> This change would help supportability, at the expense of printing a little
> more information to the log. I hope no code really depends on the output
> string.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
