zeppelin git commit: ZEPPELIN-3208. Use interpreter setting's name as its id

[zj...@apache.org]

Repository: zeppelin
Updated Branches:
  refs/heads/branch-0.8 cc2894c38 -> 97e6ef74b


ZEPPELIN-3208. Use interpreter setting's name as its id

### What is this PR for?
Interpreter setting's name is unique, so it is not necessary to use id to identify it. For now we use a random string to represent such id which is not easy to read especially when you read log for diagnose. But for backward compatibility, I will keep id and set it as the same as interpreter setting's name

### What type of PR is it?
[Improvement]

### Todos
* [ ] - Task

### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-3208

### How should this be tested?
* CI pass

### Screenshots (if appropriate)

### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No

Author: Jeff Zhang <zj...@apache.org>

Closes #2778 from zjffdu/ZEPPELIN-3208 and squashes the following commits:

56a33c2 [Jeff Zhang] ZEPPELIN-3208. Use interpreter setting's name as its id

(cherry picked from commit b3ba458bd22f4320162cbb4baf3bab48a6844b30)
Signed-off-by: Jeff Zhang <zj...@apache.org>


Project: http://git-wip-us.apache.org/repos/asf/zeppelin/repo
Commit: http://git-wip-us.apache.org/repos/asf/zeppelin/commit/97e6ef74
Tree: http://git-wip-us.apache.org/repos/asf/zeppelin/tree/97e6ef74
Diff: http://git-wip-us.apache.org/repos/asf/zeppelin/diff/97e6ef74

Branch: refs/heads/branch-0.8
Commit: 97e6ef74b07ef9204d3e4d7023e14d603b9d33d3
Parents: cc2894c
Author: Jeff Zhang <zj...@apache.org>
Authored: Wed Feb 7 18:25:46 2018 +0800
Committer: Jeff Zhang <zj...@apache.org>
Committed: Mon Mar 19 09:48:28 2018 +0800

----------------------------------------------------------------------
 .../java/org/apache/zeppelin/interpreter/InterpreterSetting.java  | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/zeppelin/blob/97e6ef74/zeppelin-zengine/src/main/java/org/apache/zeppelin/interpreter/InterpreterSetting.java
----------------------------------------------------------------------
diff --git a/zeppelin-zengine/src/main/java/org/apache/zeppelin/interpreter/InterpreterSetting.java b/zeppelin-zengine/src/main/java/org/apache/zeppelin/interpreter/InterpreterSetting.java
index b63c98a..a3b7f8d 100644
--- a/zeppelin-zengine/src/main/java/org/apache/zeppelin/interpreter/InterpreterSetting.java
+++ b/zeppelin-zengine/src/main/java/org/apache/zeppelin/interpreter/InterpreterSetting.java
@@ -261,6 +261,7 @@ public class InterpreterSetting {
 
   void postProcessing() {
     this.status = Status.READY;
+    this.id = this.name;
     if (this.lifecycleManager == null) {
       this.lifecycleManager = new NullLifecycleManager(conf);
     }
@@ -280,7 +281,7 @@ public class InterpreterSetting {
    */
   public InterpreterSetting(InterpreterSetting o) {
     this();
-    this.id = generateId();
+    this.id = o.name;
     this.name = o.name;
     this.group = o.group;
     this.properties = convertInterpreterProperties(