zeppelin-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From zjf...@apache.org
Subject zeppelin git commit: [ZEPPELIN-3049] Add the notebook id and user name to the paragraph running log
Date Wed, 15 Nov 2017 07:07:48 GMT
Repository: zeppelin
Updated Branches:
  refs/heads/master 382479fd5 -> 951544a74


[ZEPPELIN-3049] Add the notebook id and user name to the paragraph running log

### What is this PR for?
Add the notebook id and user name to the paragraph running log so that Zeppelin administrators
can check who ran which notebook's paragraph.

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

### Todos

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

### How should this be tested?
* Tested manually
* I confirmed that the following logs are printed:
    * `INFO [2017-11-14 17:19:36,917] ({pool-2-thread-2} Paragraph.java[jobRun]:381) - Run
paragraph [paragraph_id: 20171114-171822_2006383248, interpreter: md, note_id: 2CZQ23DM7,
user: user1]`
    * `INFO [2017-11-14 17:50:59,803] ({pool-2-thread-2} Paragraph.java[jobRun]:381) - Run
paragraph [paragraph_id: 20171101-170859_1328485804, interpreter: md, note_id: 2CYVGQ2WW,
user: anonymous]`

### 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: Keiji Yoshida <kjmrknsn@gmail.com>

Closes #2667 from kjmrknsn/ZEPPELIN-3049 and squashes the following commits:

63b73fe [Keiji Yoshida] [ZEPPELIN-3049] Add the notebook id and user name to the paragraph
running log


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

Branch: refs/heads/master
Commit: 951544a74a58fafeafd4f6f974ef4db20c188513
Parents: 382479f
Author: Keiji Yoshida <kjmrknsn@gmail.com>
Authored: Tue Nov 14 16:50:55 2017 +0900
Committer: Jeff Zhang <zjffdu@apache.org>
Committed: Wed Nov 15 15:07:41 2017 +0800

----------------------------------------------------------------------
 .../src/main/java/org/apache/zeppelin/notebook/Paragraph.java     | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/zeppelin/blob/951544a7/zeppelin-zengine/src/main/java/org/apache/zeppelin/notebook/Paragraph.java
----------------------------------------------------------------------
diff --git a/zeppelin-zengine/src/main/java/org/apache/zeppelin/notebook/Paragraph.java b/zeppelin-zengine/src/main/java/org/apache/zeppelin/notebook/Paragraph.java
index a38b222..6a0c27a 100644
--- a/zeppelin-zengine/src/main/java/org/apache/zeppelin/notebook/Paragraph.java
+++ b/zeppelin-zengine/src/main/java/org/apache/zeppelin/notebook/Paragraph.java
@@ -378,7 +378,8 @@ public class Paragraph extends Job implements Cloneable, JsonSerializable
{
 
   @Override
   protected Object jobRun() throws Throwable {
-    logger.info("Run paragraph {} using {} ", getId(), intpText);
+    logger.info("Run paragraph [paragraph_id: {}, interpreter: {}, note_id: {}, user: {}]",
+            getId(), intpText, note.getId(), authenticationInfo.getUser());
     this.interpreter = getBindedInterpreter();
     if (this.interpreter == null) {
       logger.error("Can not find interpreter name " + intpText);


Mime
View raw message