[jira] [Commented] (SENTRY-1513) Avoid two splits in PathsUpdate.java

["Hadoop QA (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/SENTRY-1513?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15865440#comment-15865440 ] 

Hadoop QA commented on SENTRY-1513:
-----------------------------------

Here are the results of testing the latest attachment
https://issues.apache.org/jira/secure/attachment/12843092/SENTRY-1513.002.patch against master.

{color:green}Overall:{color} +1 all checks pass

{color:green}SUCCESS:{color} all tests passed

Console output: https://builds.apache.org/job/PreCommit-SENTRY-Build/2338/console

This message is automatically generated.

> Avoid two splits in PathsUpdate.java
> ------------------------------------
>
>                 Key: SENTRY-1513
>                 URL: https://issues.apache.org/jira/browse/SENTRY-1513
>             Project: Sentry
>          Issue Type: Improvement
>            Reporter: Sravya Tirukkovalur
>            Assignee: Jan Hentschel
>            Priority: Trivial
>              Labels: newbie
>         Attachments: SENTRY-1513.001.patch, SENTRY-1513.002.patch
>
>
> {code}
> if(uriPath.split("^/").length < 2) {
> 	          throw new SentryMalformedPathException("Path part of uri does not seem right, was expecting a non empty path" +
> 	                  ": path = " + uriPath + ", uri=" + uri);
> 	        }
> 	        return Lists.newArrayList(uriPath.split("^/")[1].split("/"));
> {code}
> Also, correct the spelling in the same file
> URIs with non hdfs *schemee* will just be ignore



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)