You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@zeppelin.apache.org by zj...@apache.org on 2017/11/15 07:07:48 UTC
zeppelin git commit: [ZEPPELIN-3049] Add the notebook id and user
name to the paragraph running log
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 <kj...@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 <kj...@gmail.com>
Authored: Tue Nov 14 16:50:55 2017 +0900
Committer: Jeff Zhang <zj...@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);