jpmcmu commented on a change in pull request #3415: [ZEPPELIN-1070]: Inject
Credentials in any Interpreter-Code - 0.8x
URL: https://github.com/apache/zeppelin/pull/3415#discussion_r308399349
##########
File path:
zeppelin-zengine/src/main/java/org/apache/zeppelin/notebook/Paragraph.java
##########
@@ -434,7 +434,12 @@ protected Object jobRun() throws Throwable {
try {
InterpreterContext context = getInterpreterContext();
InterpreterContext.set(context);
- InterpreterResult ret = interpreter.interpret(script, context);
+ UserCredentials creds =
context.getAuthenticationInfo().getUserCredentials();
Review comment:
I will work on adding that in. Would it make sense to have the property be
an interpreter config setting as well? I am hoping to get this PR into both
0.8.X and 0.9 series, and currently 0.8.X does not support local properties,
but I think it makes sense to have credential injection be opt-in.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
