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 2014/06/22 22:46:25 UTC
[jira] [Created] (HDFS-6588) Investigating removing getTrueCause
method in Server.java
Yongjun Zhang created HDFS-6588:
-----------------------------------
Summary: Investigating removing getTrueCause method in Server.java
Key: HDFS-6588
URL: https://issues.apache.org/jira/browse/HDFS-6588
Project: Hadoop HDFS
Issue Type: Bug
Components: security, webhdfs
Affects Versions: 2.5.0
Reporter: Yongjun Zhang
When addressing Daryn Sharp's comment for HDFS-6475 quoted below:
{quote}
What I'm saying is I think the patch adds too much unnecessary code. Filing an improvement to delete all but a few lines of the code changed in this patch seems a bit odd. I think you just need to:
- Delete getTrueCause entirely instead of moving it elsewhere
- In saslProcess, just throw the exception instead of running it through getTrueCause since it's not a "InvalidToken wrapping another exception" anymore.
- Keep your 3-line change to unwrap SecurityException in toResponse
{quote}
There are multiple test failures, after making the suggested changes, Filing this jira to dedicate to the investigation of removing getTrueCause method.
More detail will be put in the first comment.
--
This message was sent by Atlassian JIRA
(v6.2#6252)