You are viewing a plain text version of this content. The canonical link for it is here.
Posted to hdfs-dev@hadoop.apache.org by "Yongjun Zhang (JIRA)" <ji...@apache.org> on 2016/03/25 00:31:25 UTC
[jira] [Resolved] (HDFS-10211) Add more info to
DelegationTokenIdentifier#toString for better supportability
[ https://issues.apache.org/jira/browse/HDFS-10211?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Yongjun Zhang resolved HDFS-10211.
----------------------------------
Resolution: Duplicate
> 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)