[
https://issues.apache.org/jira/browse/SENTRY-1002?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15094399#comment-15094399
]
Lenni Kuff commented on SENTRY-1002:
------------------------------------
Thanks [~rnpridgeon], that makes sense. The change looks good. Can you just add
an E2E test for the case where the path is "relative" (does not contain the
filesystem info)? I assume the behavior in that case will be to skip the path
rather than fail with a NPE.
> PathsUpdate.parsePath(path) will throw an NPE when parsing relative paths
> -------------------------------------------------------------------------
>
> Key: SENTRY-1002
> URL: https://issues.apache.org/jira/browse/SENTRY-1002
> Project: Sentry
> Issue Type: Bug
> Components: Sentry
> Affects Versions: 1.6.0
> Reporter: Ryan P
> Assignee: Ryan P
> Attachments: SENTRY-1002.patch, SENTRY-1002.patch
>
>
> There a few edge cases where Paths update may be passed a path which lacks a
> URI Scheme.
> i.e
> /user/hive/warehouse
> In this situation PathsUpdate will throw an uncaught NPE at the Precondition
> check.
> Since we only support HDFS anyway I suggest we silently ignore these relative
> paths.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
