-
Notifications
You must be signed in to change notification settings - Fork 2.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[DOCS] Add doc about zeppelin.notebook.public #1773
Conversation
@AhyoungRyu Please help review |
Push another commit to fix the double checking lock for singleton |
LGTM |
Looks good :) |
@zjffdu All CI build jobs are red now. Could you check again? |
@@ -65,7 +65,7 @@ | |||
|
|||
private NotebookAuthorization() {} | |||
|
|||
public static NotebookAuthorization init(ZeppelinConfiguration config) { | |||
public static synchronized NotebookAuthorization init(ZeppelinConfiguration config) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for a change like this perhaps it's better to have a JIRA to track change?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @felixcheung & @AhyoungRyu I have reverted and will do it in another ticket.
@zjffdu Do you have any chances to look into the last comment by @felixcheung ? |
This reverts commit 92472f9.
Looks good! Thanks @zjffdu |
What is this PR for?
Add document about zeppelin.notebook.public
What type of PR is it?
[Documentation]
What is the Jira issue?
Questions: