From de59104b689f41c3e346eefb945eafd97f0b09bf Mon Sep 17 00:00:00 2001
From: Rohit Sinha <rsinha@cask.co>
Date: Tue, 29 Aug 2017 22:31:33 -0700
Subject: [PATCH] CDAP-12148 Fix namespace lookup

---
 .../ranger/lookup/client/CDAPRangerLookupClient.java | 12 ++++++++----
 cdap-ranger/pom.xml                                  |  2 +-
 2 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/cdap-ranger/cdap-ranger-lookup/src/main/java/co/cask/cdap/security/authorization/ranger/lookup/client/CDAPRangerLookupClient.java b/cdap-ranger/cdap-ranger-lookup/src/main/java/co/cask/cdap/security/authorization/ranger/lookup/client/CDAPRangerLookupClient.java
index b7362a14..0160c62c 100644
--- a/cdap-ranger/cdap-ranger-lookup/src/main/java/co/cask/cdap/security/authorization/ranger/lookup/client/CDAPRangerLookupClient.java
+++ b/cdap-ranger/cdap-ranger-lookup/src/main/java/co/cask/cdap/security/authorization/ranger/lookup/client/CDAPRangerLookupClient.java
@@ -198,10 +198,14 @@ public List<String> call() {
               // note that in the the list calls resourceMap is used to exclude entities which has already been added
               // to the list being displayed to the user as an option for selection.
               List<String> list = null;
-              NamespaceId namespace =
-                resourceMap.containsKey(RangerCommon.KEY_NAMESPACE) &&
-                  !resourceMap.get(RangerCommon.KEY_NAMESPACE).isEmpty() ?
-                  new NamespaceId(resourceMap.get(RangerCommon.KEY_NAMESPACE).get(0)) : null;
+              NamespaceId namespace = null;
+              // if user is still entering the namespace the resourceMap.get(RangerCommon.KEY_NAMESPACE).get(0) will
+              // be empty string in that case we don't want to initialize the namespaceId
+              if (resourceMap.containsKey(RangerCommon.KEY_NAMESPACE) &&
+                !resourceMap.get(RangerCommon.KEY_NAMESPACE).isEmpty() &&
+                !resourceMap.get(RangerCommon.KEY_NAMESPACE).get(0).isEmpty()) {
+                namespace = new NamespaceId(resourceMap.get(RangerCommon.KEY_NAMESPACE).get(0));
+              }
               switch (resource.trim().toLowerCase()) {
                 case RangerCommon.KEY_NAMESPACE:
                   list = getNamespaces(resourceMap.get(RangerCommon.KEY_NAMESPACE));
diff --git a/cdap-ranger/pom.xml b/cdap-ranger/pom.xml
index e4cb71a8..bb41595f 100644
--- a/cdap-ranger/pom.xml
+++ b/cdap-ranger/pom.xml
@@ -71,7 +71,7 @@
 
   <properties>
     <hadoop.version>2.3.0</hadoop.version>
-    <ranger.version>0.7.1</ranger.version>
+    <ranger.version>0.7.0</ranger.version>
   </properties>
 
   <repositories>