[GitHub] [zeppelin] rickchengx opened a new pull request #4172: [ZEPPELIN-5444] Allow users to set service account for interpreter

2021-07-12 Thread GitBox
rickchengx opened a new pull request #4172: URL: https://github.com/apache/zeppelin/pull/4172 ### What is this PR for? Currently, users cannot set the service account for the interpreter pod under k8s mode. And there is no service account defined in the pod `spec` in `k8s/interpreter/10

[GitHub] [zeppelin] zjffdu opened a new pull request #4173: [ZEPPELIN-5455] Spark conf can not contain white space

2021-07-12 Thread GitBox
zjffdu opened a new pull request #4173: URL: https://github.com/apache/zeppelin/pull/4173 ### What is this PR for? This PR would use `|` as parameter delimiter, although it can not resolve the issue fundamentally, because parameter may still contain `|`, but it would mitigate the is

[GitHub] [zeppelin] asfgit closed pull request #4141: [ZEPPELIN-5405] ClassNotFoundException in YarnAppMonitor when hadoop client is not installed

2021-07-12 Thread GitBox
asfgit closed pull request #4141: URL: https://github.com/apache/zeppelin/pull/4141 -- 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: dev-unsubscr..

[jira] [Created] (ZEPPELIN-5457) I can't get role with ldap

2021-07-12 Thread ighack (Jira)
ighack created ZEPPELIN-5457: Summary: I can't get role with ldap Key: ZEPPELIN-5457 URL: https://issues.apache.org/jira/browse/ZEPPELIN-5457 Project: Zeppelin Issue Type: Task Componen

[GitHub] [zeppelin] Reamer commented on a change in pull request #4174: [ZEPPELIN-5442] Allow users to set a separate namespace for the interpreter

2021-07-12 Thread GitBox
Reamer commented on a change in pull request #4174: URL: https://github.com/apache/zeppelin/pull/4174#discussion_r667929497 ## File path: zeppelin-plugins/launcher/k8s-standard/src/main/java/org/apache/zeppelin/interpreter/launcher/K8sStandardInterpreterLauncher.java ## @@ -73

[GitHub] [zeppelin] Reamer commented on pull request #4171: [ZEPPELIN-5456] Allow users to set the config 'imagePullPolicy' under k8s mode

2021-07-12 Thread GitBox
Reamer commented on pull request #4171: URL: https://github.com/apache/zeppelin/pull/4171#issuecomment-878288842 I don't think we need to include all the configuration options in our example `100-interpreter-spec.yaml`. This file should be configured by the user. Maybe we should write mo

[GitHub] [zeppelin] Reamer commented on a change in pull request #4169: [ZEPPELIN-5454] Explain output should be in text format

2021-07-12 Thread GitBox
Reamer commented on a change in pull request #4169: URL: https://github.com/apache/zeppelin/pull/4169#discussion_r667951650 ## File path: spark/spark1-shims/src/main/scala/org/apache/zeppelin/spark/Spark1Shims.java ## @@ -81,17 +82,28 @@ public String showDataFrame(Object obj,

[GitHub] [zeppelin] zjffdu commented on pull request #4080: [ZEPPELIN-5300] Paragraph pending until timeout when process launcher has already failed

2021-07-12 Thread GitBox
zjffdu commented on pull request #4080: URL: https://github.com/apache/zeppelin/pull/4080#issuecomment-878120436 @Reamer Sorry for response late, the root cause is in the yarn-cluster mode, the launcher process may transition to COMPLETED state. Now I fixed it in this PR -- This is an

[GitHub] [zeppelin] Reamer commented on pull request #4173: [ZEPPELIN-5455] Spark conf can not contain white space

2021-07-12 Thread GitBox
Reamer commented on pull request #4173: URL: https://github.com/apache/zeppelin/pull/4173#issuecomment-878124118 @zjffdu Please take a look at #4127 and my [comment](https://github.com/apache/zeppelin/pull/4127#issuecomment-864825992). I tried to remove IFS, which is not correct for wo

[GitHub] [zeppelin] zjffdu edited a comment on pull request #4080: [ZEPPELIN-5300] Paragraph pending until timeout when process launcher has already failed

2021-07-12 Thread GitBox
zjffdu edited a comment on pull request #4080: URL: https://github.com/apache/zeppelin/pull/4080#issuecomment-878120436 -- 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 u

[GitHub] [zeppelin] zjffdu commented on pull request #4127: ZEPPELIN-5397 : Modified the interpreter.sh script to be able to parse SPARK_SUBMIT_OPTIONS

2021-07-12 Thread GitBox
zjffdu commented on pull request #4127: URL: https://github.com/apache/zeppelin/pull/4127#issuecomment-878129113 > export SPARK_SUBMIT_OPTIONS="$SPARK_SUBMIT_OPTIONS --conf 'spark.driver.extraJavaOptions=-Dcom.example.env=production -Dcom.example.role=zeppelin' --conf mytest=myvalue" >

[GitHub] [zeppelin] Reamer commented on pull request #4172: [ZEPPELIN-5444] Allow users to set service account for interpreter

2021-07-12 Thread GitBox
Reamer commented on pull request #4172: URL: https://github.com/apache/zeppelin/pull/4172#issuecomment-878285741 Please a documentation. Missing parts: - How to create the service account -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [zeppelin] zjffdu commented on a change in pull request #4169: [ZEPPELIN-5454] Explain output should be in text format

2021-07-12 Thread GitBox
zjffdu commented on a change in pull request #4169: URL: https://github.com/apache/zeppelin/pull/4169#discussion_r667966841 ## File path: spark/spark1-shims/src/main/scala/org/apache/zeppelin/spark/Spark1Shims.java ## @@ -81,17 +82,28 @@ public String showDataFrame(Object obj,

[GitHub] [zeppelin] zjffdu commented on pull request #4134: [ZEPPELIN-5398] fix ZEPPELIN-5398, make corrupted notes deletable

2021-07-12 Thread GitBox
zjffdu commented on pull request #4134: URL: https://github.com/apache/zeppelin/pull/4134#issuecomment-878141005 LGTM -- 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 un

[GitHub] [zeppelin] Reamer commented on pull request #4170: [ZEPPELIN-5360] K8S interpreter create spark pods, should support the 'imagePullSecrets' config

2021-07-12 Thread GitBox
Reamer commented on pull request #4170: URL: https://github.com/apache/zeppelin/pull/4170#issuecomment-878310927 Same as in #4171 > I don't think we need to include all the configuration options in our example 100-interpreter-spec.yaml. This file should be configured by the user.

[GitHub] [zeppelin] Reamer edited a comment on pull request #4172: [ZEPPELIN-5444] Allow users to set service account for interpreter

2021-07-12 Thread GitBox
Reamer edited a comment on pull request #4172: URL: https://github.com/apache/zeppelin/pull/4172#issuecomment-878285741 Please add documentation. Missing parts: - How to create the service account -- This is an automated message from the Apache Git Service. To respond to the messag

[GitHub] [zeppelin] Reamer commented on a change in pull request #4134: [ZEPPELIN-5398] fix ZEPPELIN-5398, make corrupted notes deletable

2021-07-12 Thread GitBox
Reamer commented on a change in pull request #4134: URL: https://github.com/apache/zeppelin/pull/4134#discussion_r667834673 ## File path: zeppelin-server/src/test/java/org/apache/zeppelin/service/NotebookServiceTest.java ## @@ -263,6 +264,22 @@ public void testNoteOperations()

[GitHub] [zeppelin] rickchengx commented on pull request #4172: [ZEPPELIN-5444] Allow users to set service account for interpreter

2021-07-12 Thread GitBox
rickchengx commented on pull request #4172: URL: https://github.com/apache/zeppelin/pull/4172#issuecomment-878735434 > Please add documentation. > Missing parts: > > * How to create the service account If user needs to set a specific service account (rather than default), he

[jira] [Created] (ZEPPELIN-5458) zepplin can not parse columns correctly which contains chinese character

2021-07-12 Thread zhangxiaoping (Jira)
zhangxiaoping created ZEPPELIN-5458: --- Summary: zepplin can not parse columns correctly which contains chinese character Key: ZEPPELIN-5458 URL: https://issues.apache.org/jira/browse/ZEPPELIN-5458 Pr

[GitHub] [zeppelin] rickchengx commented on pull request #4171: [ZEPPELIN-5456] Allow users to set the config 'imagePullPolicy' under k8s mode

2021-07-12 Thread GitBox
rickchengx commented on pull request #4171: URL: https://github.com/apache/zeppelin/pull/4171#issuecomment-878745664 > I don't think we need to include all the configuration options in our example `100-interpreter-spec.yaml`. This file should be configured by the user. > Maybe we should

[GitHub] [zeppelin] rickchengx commented on pull request #4170: [ZEPPELIN-5360] K8S interpreter create spark pods, should support the 'imagePullSecrets' config

2021-07-12 Thread GitBox
rickchengx commented on pull request #4170: URL: https://github.com/apache/zeppelin/pull/4170#issuecomment-878747422 > Same as in #4171 > > > I don't think we need to include all the configuration options in our example 100-interpreter-spec.yaml. This file should be configured by the

[GitHub] [zeppelin] Reamer commented on pull request #4172: [ZEPPELIN-5444] Allow users to set service account for interpreter

2021-07-12 Thread GitBox
Reamer commented on pull request #4172: URL: https://github.com/apache/zeppelin/pull/4172#issuecomment-878789869 Please expand the [documentation](https://github.com/apache/zeppelin/blob/master/docs/quickstart/kubernetes.md) about this feature. -- This is an automated message from the A

[GitHub] [zeppelin] Reamer commented on pull request #4171: [ZEPPELIN-5456] Allow users to set the config 'imagePullPolicy' under k8s mode

2021-07-12 Thread GitBox
Reamer commented on pull request #4171: URL: https://github.com/apache/zeppelin/pull/4171#issuecomment-878790651 Please expand the documentation on this function so that new users know about it and how to use it. -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [zeppelin] Reamer commented on pull request #4170: [ZEPPELIN-5360] K8S interpreter create spark pods, should support the 'imagePullSecrets' config

2021-07-12 Thread GitBox
Reamer commented on pull request #4170: URL: https://github.com/apache/zeppelin/pull/4170#issuecomment-878790798 Please expand the documentation on this function so that new users know about it and how to use it. -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [zeppelin] Reamer commented on pull request #4170: [ZEPPELIN-5360] K8S interpreter create spark pods, should support the 'imagePullSecrets' config

2021-07-12 Thread GitBox
Reamer commented on pull request #4170: URL: https://github.com/apache/zeppelin/pull/4170#issuecomment-878794935 > Especially when the user directly runs the zeppelin server in the k8s cluster, every time the `100-interpreter-spec.yaml` file is modified, the docker image needs to be rebuil