Skip to content

AMBARI-22698: Custom zeppelin interpreter properties are getting removed after moving zeppelin to a different host#162

Merged
nitirajrathore merged 1 commit into
apache:trunkfrom
prabhjyotsingh:AMBARI-22698
Jan 22, 2018
Merged

AMBARI-22698: Custom zeppelin interpreter properties are getting removed after moving zeppelin to a different host#162
nitirajrathore merged 1 commit into
apache:trunkfrom
prabhjyotsingh:AMBARI-22698

Conversation

@prabhjyotsingh
Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Custom zeppelin interpreter properties are getting removed after moving zeppelin to a different host

How was this patch tested?

Manually
Custom zeppelin interpreter properties are getting lost after moving zeppelin to a different host

Steps to reproduce :

  1. Modify interpreter settings
    a) Add new custom properties
    b) Modify existing properties
    Changes are getting saved in HDFS path /user/zeppelin/conf/interpreter.json and they are getting synced to local path /etc/zeppelin/conf/interpreter.json after zeppelin restart.
  2. Delete zeppelin server
  3. Install zeppelin server on a different host
  4. Modifications done in step 1 are lost.

…ved after moving zeppelin to a different host (prabhjyotsingh)
@prabhjyotsingh
Copy link
Copy Markdown
Contributor Author

@venkatasairamlanka please help review/merge this in trunk/branch-2.6
ref: https://reviews.apache.org/r/64861/

@asfgit
Copy link
Copy Markdown

asfgit commented Jan 22, 2018

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/224/
Test PASSed.

@nitirajrathore
Copy link
Copy Markdown
Contributor

Will merge this based on reviews on https://reviews.apache.org/r/64861/

@nitirajrathore nitirajrathore merged commit a027d91 into apache:trunk Jan 22, 2018
@prabhjyotsingh prabhjyotsingh deleted the AMBARI-22698 branch August 1, 2024 21:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants