Reamer commented on PR #5095:
URL: https://github.com/apache/zeppelin/pull/5095#issuecomment-3409482135

   > Why is `ZeppelinConfiguration` in the `zeppelin-interpreter` module?
   
   In my opinion, we should split this up. ZeppelinConfiguration belongs to the 
Zeppelin server, and the Zeppelin interpreter should really only work on a 
HashMap with ConfigKey and ConfigValue.
   
   The entire loading of the configuration and possibly also a reload should 
all belong to the server.
   
   Currently, the Zeppelin server transmits the configuration from the server 
to the remote interpreter when it starts up. See 
https://github.com/apache/zeppelin/blob/a03dcbcefc4e6d36993fd537d41626e27eb8e05a/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/remote/RemoteInterpreterServer.java#L202-L207


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to