[
https://issues.apache.org/jira/browse/HADOOP-7531?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13082746#comment-13082746
]
Todd Lipcon commented on HADOOP-7531:
-------------------------------------
hmm:
{code}
+
Preconditions.checkArgument(request.getRequestURI().startsWith(servletName));
+ return null == request.getRequestURI() ? "/" :
{code}
The second line implies that request.getRequestURI() may be null, in which case
the first line would throw NPE. Is it actually possible to be null? If so,
separate the ternary into an if/else with the precondition inside the if, I
guess?
> Add servlet util methods for handling paths in requests
> --------------------------------------------------------
>
> Key: HADOOP-7531
> URL: https://issues.apache.org/jira/browse/HADOOP-7531
> Project: Hadoop Common
> Issue Type: Improvement
> Components: util
> Affects Versions: 0.23.0
> Reporter: Eli Collins
> Assignee: Eli Collins
> Fix For: 0.23.0
>
> Attachments: hadoop-7531-1.patch, hadoop-7531-1.patch,
> hadoop-7531-2.patch, hadoop-7531-3.patch
>
>
> Common side of HDFS-2235.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
