sentry-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From s..@apache.org
Subject [12/25] incubator-sentry git commit: SENTRY-892: parsePath should handle empty paths well ( Sravya Tirukkovalur, Reviewed by: Lenni Kuff and Colin Ma)
Date Thu, 05 Nov 2015 01:44:40 GMT
SENTRY-892: parsePath should handle empty paths well ( Sravya Tirukkovalur, Reviewed by: Lenni
Kuff and Colin Ma)


Project: http://git-wip-us.apache.org/repos/asf/incubator-sentry/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-sentry/commit/45b28a77
Tree: http://git-wip-us.apache.org/repos/asf/incubator-sentry/tree/45b28a77
Diff: http://git-wip-us.apache.org/repos/asf/incubator-sentry/diff/45b28a77

Branch: refs/heads/hive_plugin_v2
Commit: 45b28a775b646f58e357c7fcdfa248fd33967406
Parents: e8723d0
Author: Sravya Tirukkovalur <sravya@cloudera.com>
Authored: Tue Sep 22 09:31:25 2015 -0500
Committer: Sun Dapeng <sdp@apache.org>
Committed: Mon Nov 2 16:36:30 2015 +0800

----------------------------------------------------------------------
 .../src/main/java/org/apache/sentry/hdfs/PathsUpdate.java        | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-sentry/blob/45b28a77/sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/PathsUpdate.java
----------------------------------------------------------------------
diff --git a/sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/PathsUpdate.java
b/sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/PathsUpdate.java
index 79019f4..8c5edd7 100644
--- a/sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/PathsUpdate.java
+++ b/sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/PathsUpdate.java
@@ -91,7 +91,7 @@ public class PathsUpdate implements Updateable.Update {
    *
    * @param path : Needs to be a HDFS location with scheme
    * @return Path in the form a list containing the path tree with scheme/ authority stripped
off.
-   * Returns null if a non HDFS path
+   * Returns null if a non HDFS path or if path is null/empty
    */
   public static List<String> parsePath(String path) {
     try {
@@ -99,6 +99,8 @@ public class PathsUpdate implements Updateable.Update {
       URI uri = null;
       if (StringUtils.isNotEmpty(path)) {
         uri = new URI(URIUtil.encodePath(path));
+      } else {
+        return null;
       }
 
       Preconditions.checkNotNull(uri.getScheme());


Mime
View raw message