You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zeppelin.apache.org by GitBox <gi...@apache.org> on 2019/11/04 14:07:00 UTC

[GitHub] [zeppelin] zjffdu opened a new pull request #3510: [ZEPPELIN-4416]. Don't merge properties from inteprreter-setting.json if it already exists

zjffdu opened a new pull request #3510: [ZEPPELIN-4416]. Don't merge properties from inteprreter-setting.json if it already exists
URL: https://github.com/apache/zeppelin/pull/3510
 
 
   ### What is this PR for?
   
   Currently zeppelin will merge the setting in `interpreter-setting.json` with setting in `interpreter.json`, usually this is not what user expect. E.g. if user delete one property defined in `interpreter-setting.json` and then restart zeppelin, he would see this property again as we merged it from `interpreter-setting.json`. So this PR is to remove the logic of merging properties from interpreter-setting.json if it already exists.
   
   
   ### What type of PR is it?
   [Improvement]
   
   ### Todos
   * [ ] - Task
   
   ### What is the Jira issue?
   * https://jira.apache.org/jira/browse/ZEPPELIN-4416
   
   ### How should this be tested?
   * Manually tested
   
   ### Screenshots (if appropriate)
   
   ### Questions:
   * Does the licenses files need update? No
   * Is there breaking changes for older versions? No
   * Does this needs documentation? No
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services