add logger.warn for hive and impersonation

prabhjyotsingh · prabhjyotsingh · commit 84b5e55b2821 · 2017-04-30T00:15:19.000+05:30
diff --git a/docs/interpreter/jdbc.md b/docs/interpreter/jdbc.md
@@ -538,7 +538,7 @@ When Zeppelin server is running with authentication enabled, then the interprete
 
 To enable this set following:
   - `zeppelin.jdbc.auth.type` as `SIMPLE` or `KERBEROS` (if required) in the interpreter setting.
-  - `{propertyKey}.proxy.user.property` as `hive.server2.proxy.user`
+  - `${prefix}.proxy.user.property` as `hive.server2.proxy.user`
   Example configuration
 
   *Properties*
diff --git a/jdbc/src/main/java/org/apache/zeppelin/jdbc/JDBCInterpreter.java b/jdbc/src/main/java/org/apache/zeppelin/jdbc/JDBCInterpreter.java
@@ -413,6 +413,9 @@ private String appendProxyUserToURL(String url, String user, String propertyKey)
           basePropretiesMap.get(propertyKey).getProperty("proxy.user.property"));
       connectionUrl.insert(lastIndexOfUrl, ";" +
           basePropretiesMap.get(propertyKey).getProperty("proxy.user.property") + "=" + user + ";");
+    } else if (user != null && !user.equals("anonymous") && url.contains("hive")) {
+      logger.warn("User impersonation for hive has changed please refer: http://zeppelin.apache" +
+          ".org/docs/latest/interpreter/jdbc.html#apache-hive");
     }
 
     return connectionUrl.toString();

Original file line number	Diff line number	Diff line change
`@@ -413,6 +413,9 @@ private String appendProxyUserToURL(String url, String user, String propertyKey)`
`413`	`413`	`basePropretiesMap.get(propertyKey).getProperty("proxy.user.property"));`
`414`	`414`	`connectionUrl.insert(lastIndexOfUrl, ";" +`
`415`	`415`	`basePropretiesMap.get(propertyKey).getProperty("proxy.user.property") + "=" + user + ";");`
	`416`	`+ } else if (user != null && !user.equals("anonymous") && url.contains("hive")) {`
	`417`	`+ logger.warn("User impersonation for hive has changed please refer: http://zeppelin.apache" +`
	`418`	`+ ".org/docs/latest/interpreter/jdbc.html#apache-hive");`
`416`	`419`	`}`
`417`	`420`
`418`	`421`	`return connectionUrl.toString();`