-
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
[ZEPPELIN-3655] Add missing roles information to api/notebook/job/ method #3093
Conversation
Can you add unit test ? |
@zjffdu |
@oxygen311 Could you resolve conflict ? |
bin/common.sh
Outdated
@@ -16,6 +16,10 @@ | |||
# limitations under the License. | |||
# | |||
|
|||
# -agentlib:jdwp=transport=dt_socket,server=y,suspend=n,address=5005 | |||
# ZEPPELIN_JAVA_OPTS | |||
export ZEPPELIN_JAVA_OPTS="-agentlib:jdwp=transport=dt_socket,server=y,suspend=n,address=5005" |
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.
Please remove it
…ELIN-3655 # Conflicts: # zeppelin-server/src/main/java/org/apache/zeppelin/rest/NotebookRestApi.java
@zjffdu done |
It looks like all the public api in NotebookRestApi miss the role info. Do you see any issue when roles is missing ? I think it is better to set roles properly in some kind of Filter. |
@zjffdu |
Thanks @oxygen311 LGTM |
…thod ### What is this PR for? Fix bugs with private access interpreter launched with REST by adding missing roles information to `AuthenticationInfo`. ### What type of PR is it? Bug Fix ### What is the Jira issue? [ZEPPELIN-3655](https://issues.apache.org/jira/projects/ZEPPELIN/issues/ZEPPELIN-3655) ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? No Author: oxygen311 <alex39-09@mail.ru> Closes apache#3093 from oxygen311/ZEPPELIN-3655 and squashes the following commits: aa51ca3 [oxygen311] Add roles info to run paragraph method 407b104 [oxygen311] Change 0444284 [oxygen311] Merge branch 'master' of https://github.com/apache/zeppelin into ZEPPELIN-3655 182dadc [oxygen311] Fix roles info
…thod Fix bugs with private access interpreter launched with REST by adding missing roles information to `AuthenticationInfo`. Bug Fix [ZEPPELIN-3655](https://issues.apache.org/jira/projects/ZEPPELIN/issues/ZEPPELIN-3655) * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? No Author: oxygen311 <alex39-09@mail.ru> Closes #3093 from oxygen311/ZEPPELIN-3655 and squashes the following commits: aa51ca3 [oxygen311] Add roles info to run paragraph method 407b104 [oxygen311] Change 0444284 [oxygen311] Merge branch 'master' of https://github.com/apache/zeppelin into ZEPPELIN-3655 182dadc [oxygen311] Fix roles info (cherry picked from commit 858cfcd) Signed-off-by: Jeff Zhang <zjffdu@apache.org>
What is this PR for?
Fix bugs with private access interpreter launched with REST by adding missing roles information to
AuthenticationInfo
.What type of PR is it?
Bug Fix
What is the Jira issue?
ZEPPELIN-3655
Questions: