[GitHub] [zeppelin] asfgit closed pull request #3382: ZEPPELIN-4193 Upgrade Bouncy Castle bcpkix-jdk15on to 1.60

2019-06-13 Thread GitBox
asfgit closed pull request #3382: ZEPPELIN-4193 Upgrade Bouncy Castle bcpkix-jdk15on to 1.60 URL: https://github.com/apache/zeppelin/pull/3382 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [zeppelin] r-kamath commented on issue #3382: ZEPPELIN-4193 Upgrade Bouncy Castle bcpkix-jdk15on to 1.60

2019-06-13 Thread GitBox
r-kamath commented on issue #3382: ZEPPELIN-4193 Upgrade Bouncy Castle bcpkix-jdk15on to 1.60 URL: https://github.com/apache/zeppelin/pull/3382#issuecomment-501962711 will merge this if there are no more discussions This is

[GitHub] [zeppelin] r-kamath commented on issue #3379: [ZEPPELIN-4188] Upgrade Jetty to 9.4.18.v20190429

2019-06-13 Thread GitBox
r-kamath commented on issue #3379: [ZEPPELIN-4188] Upgrade Jetty to 9.4.18.v20190429 URL: https://github.com/apache/zeppelin/pull/3379#issuecomment-501962719 will merge this if there are no more discussions This is an

[GitHub] [zeppelin] prabhjyotsingh merged pull request #3381: [ZEPPELIN-4192] Travis CI fails for dependency download

2019-06-13 Thread GitBox
prabhjyotsingh merged pull request #3381: [ZEPPELIN-4192] Travis CI fails for dependency download URL: https://github.com/apache/zeppelin/pull/3381 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [zeppelin] prabhjyotsingh merged pull request #3378: [ZEPPELIN-4187] Bump up version of Scala from 2.11.8 to 2.11.12

2019-06-13 Thread GitBox
prabhjyotsingh merged pull request #3378: [ZEPPELIN-4187] Bump up version of Scala from 2.11.8 to 2.11.12 URL: https://github.com/apache/zeppelin/pull/3378 This is an automated message from the Apache Git Service. To

Re: [GitHub] [zeppelin] liuxunorg commented on a change in pull request #3383: [ZEPPELIN-4104] Zeppelin Interpreter On Docker

2019-06-13 Thread Xun Liu
According to the shared settings of the interpreter in zeppelin, shared \ Per node \ Per user settings, Each interpreter process will be a separate docker container. > On Jun 13, 2019, at 9:51 PM, tecgi...@yahoo.com.INVALID wrote: > > I have question about the behavior. > > Suppose user uses

[GitHub] [zeppelin] prabhjyotsingh commented on issue #3378: [ZEPPELIN-4187] Bump up version of Scala from 2.11.8 to 2.11.12

2019-06-13 Thread GitBox
prabhjyotsingh commented on issue #3378: [ZEPPELIN-4187] Bump up version of Scala from 2.11.8 to 2.11.12 URL: https://github.com/apache/zeppelin/pull/3378#issuecomment-501738858 Thank you @zjffdu Will merge this, if no more discussion.

[GitHub] [zeppelin] zjffdu commented on issue #3378: [ZEPPELIN-4187] Bump up version of Scala from 2.11.8 to 2.11.12

2019-06-13 Thread GitBox
zjffdu commented on issue #3378: [ZEPPELIN-4187] Bump up version of Scala from 2.11.8 to 2.11.12 URL: https://github.com/apache/zeppelin/pull/3378#issuecomment-501723054 LGTM This is an automated message from the Apache Git

Re: [GitHub] [zeppelin] liuxunorg commented on a change in pull request #3383: [ZEPPELIN-4104] Zeppelin Interpreter On Docker

2019-06-13 Thread tecgie88
I have question about the behavior. Suppose user uses the per note scope and his notebook consists of python, r, and sql paragraphs. How many interpreter containers will be spawned when the note is run? Thanks > On Jun 13, 2019, at 3:27 AM, GitBox wrote: > > liuxunorg commented on a change

[jira] [Created] (ZEPPELIN-4194) System command injection via Spark Interpreter configuration

2019-06-13 Thread Alexey Sintsov (JIRA)
Alexey Sintsov created ZEPPELIN-4194: Summary: System command injection via Spark Interpreter configuration Key: ZEPPELIN-4194 URL: https://issues.apache.org/jira/browse/ZEPPELIN-4194 Project:

[GitHub] [zeppelin] r-kamath commented on issue #3379: [ZEPPELIN-4188] Upgrade Jetty to 9.4.18.v20190429

2019-06-13 Thread GitBox
r-kamath commented on issue #3379: [ZEPPELIN-4188] Upgrade Jetty to 9.4.18.v20190429 URL: https://github.com/apache/zeppelin/pull/3379#issuecomment-501665032 @jongyoul there are some security issues in 9.4.14.v20181114. Zeppelin is not directly affected.

[GitHub] [zeppelin] r-kamath commented on a change in pull request #3376: [ZEPPELIN-4185] Upgrade Thrift to 0.12.0

2019-06-13 Thread GitBox
r-kamath commented on a change in pull request #3376: [ZEPPELIN-4185] Upgrade Thrift to 0.12.0 URL: https://github.com/apache/zeppelin/pull/3376#discussion_r293330494 ## File path: zeppelin-interpreter/src/test/java/org/apache/zeppelin/dep/DependencyResolverTest.java ##

[GitHub] [zeppelin] prabhjyotsingh commented on issue #3378: [ZEPPELIN-4187] Bump up version of Scala from 2.11.8 to 2.11.12

2019-06-13 Thread GitBox
prabhjyotsingh commented on issue #3378: [ZEPPELIN-4187] Bump up version of Scala from 2.11.8 to 2.11.12 URL: https://github.com/apache/zeppelin/pull/3378#issuecomment-501643228 @zjffdu these two changes looks ok, right?

[GitHub] [zeppelin] prabhjyotsingh commented on issue #3381: [ZEPPELIN-4192] Travis CI fails for dependency download

2019-06-13 Thread GitBox
prabhjyotsingh commented on issue #3381: [ZEPPELIN-4192] Travis CI fails for dependency download URL: https://github.com/apache/zeppelin/pull/3381#issuecomment-501642941 CI green https://travis-ci.org/prabhjyotsingh/zeppelin/builds/545094596 Will merge this soon.

[GitHub] [zeppelin] prabhjyotsingh commented on issue #3378: [ZEPPELIN-4187] Bump up version of Scala from 2.11.8 to 2.11.12

2019-06-13 Thread GitBox
prabhjyotsingh commented on issue #3378: [ZEPPELIN-4187] Bump up version of Scala from 2.11.8 to 2.11.12 URL: https://github.com/apache/zeppelin/pull/3378#issuecomment-501596448 Sure, I've just reverted all of Flink changes as well.

[GitHub] [zeppelin] zjffdu commented on a change in pull request #3383: [ZEPPELIN-4104] Zeppelin Interpreter On Docker

2019-06-13 Thread GitBox
zjffdu commented on a change in pull request #3383: [ZEPPELIN-4104] Zeppelin Interpreter On Docker URL: https://github.com/apache/zeppelin/pull/3383#discussion_r293244947 ## File path:

[GitHub] [zeppelin] asfgit closed pull request #3372: [ZEPPELIN-3623] Create interpreter process in the cluster mode

2019-06-13 Thread GitBox
asfgit closed pull request #3372: [ZEPPELIN-3623] Create interpreter process in the cluster mode URL: https://github.com/apache/zeppelin/pull/3372 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [zeppelin] liuxunorg commented on a change in pull request #3383: [ZEPPELIN-4104] Zeppelin Interpreter On Docker

2019-06-13 Thread GitBox
liuxunorg commented on a change in pull request #3383: [ZEPPELIN-4104] Zeppelin Interpreter On Docker URL: https://github.com/apache/zeppelin/pull/3383#discussion_r293240528 ## File path:

[GitHub] [zeppelin] liuxunorg commented on a change in pull request #3383: [ZEPPELIN-4104] Zeppelin Interpreter On Docker

2019-06-13 Thread GitBox
liuxunorg commented on a change in pull request #3383: [ZEPPELIN-4104] Zeppelin Interpreter On Docker URL: https://github.com/apache/zeppelin/pull/3383#discussion_r293240332 ## File path:

[GitHub] [zeppelin] liuxunorg commented on a change in pull request #3383: [ZEPPELIN-4104] Zeppelin Interpreter On Docker

2019-06-13 Thread GitBox
liuxunorg commented on a change in pull request #3383: [ZEPPELIN-4104] Zeppelin Interpreter On Docker URL: https://github.com/apache/zeppelin/pull/3383#discussion_r293238678 ## File path:

[GitHub] [zeppelin] liuxunorg commented on issue #3372: [ZEPPELIN-3623] Create interpreter process in the cluster mode

2019-06-13 Thread GitBox
liuxunorg commented on issue #3372: [ZEPPELIN-3623] Create interpreter process in the cluster mode URL: https://github.com/apache/zeppelin/pull/3372#issuecomment-501581945 [CI failure is unrelated.](https://travis-ci.org/liuxunorg/zeppelin/builds/545043371) I have developed other PR,

[GitHub] [zeppelin] zjffdu commented on issue #3378: [ZEPPELIN-4187] Bump up version of Scala from 2.11.8 to 2.11.12

2019-06-13 Thread GitBox
zjffdu commented on issue #3378: [ZEPPELIN-4187] Bump up version of Scala from 2.11.8 to 2.11.12 URL: https://github.com/apache/zeppelin/pull/3378#issuecomment-501581486 As I mentioned before, this PR's scope should be only limited to spark module. Other module should not be affected.

[GitHub] [zeppelin] zjffdu commented on a change in pull request #3383: [ZEPPELIN-4104] Zeppelin Interpreter On Docker

2019-06-13 Thread GitBox
zjffdu commented on a change in pull request #3383: [ZEPPELIN-4104] Zeppelin Interpreter On Docker URL: https://github.com/apache/zeppelin/pull/3383#discussion_r293235298 ## File path:

[GitHub] [zeppelin] zjffdu commented on a change in pull request #3383: [ZEPPELIN-4104] Zeppelin Interpreter On Docker

2019-06-13 Thread GitBox
zjffdu commented on a change in pull request #3383: [ZEPPELIN-4104] Zeppelin Interpreter On Docker URL: https://github.com/apache/zeppelin/pull/3383#discussion_r293235311 ## File path:

[GitHub] [zeppelin] zjffdu commented on a change in pull request #3383: [ZEPPELIN-4104] Zeppelin Interpreter On Docker

2019-06-13 Thread GitBox
zjffdu commented on a change in pull request #3383: [ZEPPELIN-4104] Zeppelin Interpreter On Docker URL: https://github.com/apache/zeppelin/pull/3383#discussion_r293234986 ## File path:

[GitHub] [zeppelin] felixcheung commented on issue #3378: [ZEPPELIN-4187] Bump up version of Scala from 2.11.8 to 2.11.12

2019-06-13 Thread GitBox
felixcheung commented on issue #3378: [ZEPPELIN-4187] Bump up version of Scala from 2.11.8 to 2.11.12 URL: https://github.com/apache/zeppelin/pull/3378#issuecomment-501580260 it's definitely strange that the flink interpreter is depending on spark eg. `org.apache.spark.repl`

[GitHub] [zeppelin] felixcheung commented on a change in pull request #3380: ZEPPELIN-4154. Build docker image for each interpreter

2019-06-13 Thread GitBox
felixcheung commented on a change in pull request #3380: ZEPPELIN-4154. Build docker image for each interpreter URL: https://github.com/apache/zeppelin/pull/3380#discussion_r293234388 ## File path: scripts/docker/zeppelin-interpreter/Dockerfile_base ## @@ -0,0 +1,53 @@ +#

[GitHub] [zeppelin] liuxunorg commented on issue #3383: [ZEPPELIN-4104] Zeppelin Interpreter On Docker

2019-06-13 Thread GitBox
liuxunorg commented on issue #3383: [ZEPPELIN-4104] Zeppelin Interpreter On Docker URL: https://github.com/apache/zeppelin/pull/3383#issuecomment-501576748 @zjffdu , Please help me review code, Thanks! This is an automated

[GitHub] [zeppelin] liuxunorg opened a new pull request #3383: [ZEPPELIN-4104] Zeppelin Interpreter On Docker

2019-06-13 Thread GitBox
liuxunorg opened a new pull request #3383: [ZEPPELIN-4104] Zeppelin Interpreter On Docker URL: https://github.com/apache/zeppelin/pull/3383 ### What is this PR for? Zeppelin service runs on local server. zeppelin is able to run the interpreter in the docker container, Isolating the

[GitHub] [zeppelin] prabhjyotsingh commented on a change in pull request #3381: [ZEPPELIN-4192] Travis CI fails for dependency download

2019-06-13 Thread GitBox
prabhjyotsingh commented on a change in pull request #3381: [ZEPPELIN-4192] Travis CI fails for dependency download URL: https://github.com/apache/zeppelin/pull/3381#discussion_r293230843 ## File path:

[GitHub] [zeppelin] zjffdu commented on a change in pull request #3381: [ZEPPELIN-4192] Travis CI fails for dependency download

2019-06-13 Thread GitBox
zjffdu commented on a change in pull request #3381: [ZEPPELIN-4192] Travis CI fails for dependency download URL: https://github.com/apache/zeppelin/pull/3381#discussion_r293228626 ## File path: zeppelin-interpreter/src/test/java/org/apache/zeppelin/dep/DependencyResolverTest.java

[GitHub] [zeppelin] prabhjyotsingh commented on a change in pull request #3381: [ZEPPELIN-4192] Travis CI fails for dependency download

2019-06-13 Thread GitBox
prabhjyotsingh commented on a change in pull request #3381: [ZEPPELIN-4192] Travis CI fails for dependency download URL: https://github.com/apache/zeppelin/pull/3381#discussion_r293227980 ## File path:

[GitHub] [zeppelin] zjffdu commented on a change in pull request #3381: [ZEPPELIN-4192] Travis CI fails for dependency download

2019-06-13 Thread GitBox
zjffdu commented on a change in pull request #3381: [ZEPPELIN-4192] Travis CI fails for dependency download URL: https://github.com/apache/zeppelin/pull/3381#discussion_r293224271 ## File path: zeppelin-interpreter/src/test/java/org/apache/zeppelin/dep/DependencyResolverTest.java

[GitHub] [zeppelin] prabhjyotsingh commented on issue #3381: [ZEPPELIN-4192] Travis CI fails for dependency download

2019-06-13 Thread GitBox
prabhjyotsingh commented on issue #3381: [ZEPPELIN-4192] Travis CI fails for dependency download URL: https://github.com/apache/zeppelin/pull/3381#issuecomment-501567089 @r-kamath @zjffdu can you help review this? If this is OK we can merge this soon, as this may affect other PRs as

[GitHub] [zeppelin] r-kamath opened a new pull request #3382: ZEPPELIN-4193 Upgrade Bouncy Castle bcpkix-jdk15on to 1.60

2019-06-13 Thread GitBox
r-kamath opened a new pull request #3382: ZEPPELIN-4193 Upgrade Bouncy Castle bcpkix-jdk15on to 1.60 URL: https://github.com/apache/zeppelin/pull/3382 ### What is this PR for? Upgrade Bouncy Castle bcpkix-jdk15on to 1.60 ### What type of PR is it? Improvement ### Todos

[jira] [Created] (ZEPPELIN-4193) Upgrade Bouncy Castle bcpkix-jdk15on to 1.60

2019-06-13 Thread Renjith Kamath (JIRA)
Renjith Kamath created ZEPPELIN-4193: Summary: Upgrade Bouncy Castle bcpkix-jdk15on to 1.60 Key: ZEPPELIN-4193 URL: https://issues.apache.org/jira/browse/ZEPPELIN-4193 Project: Zeppelin

[GitHub] [zeppelin] prabhjyotsingh opened a new pull request #3381: [ZEPPELIN-4192] Travis CI fails for dependency download

2019-06-13 Thread GitBox
prabhjyotsingh opened a new pull request #3381: [ZEPPELIN-4192] Travis CI fails for dependency download URL: https://github.com/apache/zeppelin/pull/3381 ### What is this PR for? Travis CI is failing with "DependencyResolverTest.testLoad:92 » Repository Cannot fetch dependencies for

[GitHub] [zeppelin] jongyoul commented on issue #3379: [ZEPPELIN-4188] Upgrade Jetty to 9.4.18.v20190429

2019-06-13 Thread GitBox
jongyoul commented on issue #3379: [ZEPPELIN-4188] Upgrade Jetty to 9.4.18.v20190429 URL: https://github.com/apache/zeppelin/pull/3379#issuecomment-501561330 What kind of potential issue will we have? This is an automated

[jira] [Created] (ZEPPELIN-4192) Travis CI fails for dependency download

2019-06-13 Thread Prabhjyot Singh (JIRA)
Prabhjyot Singh created ZEPPELIN-4192: - Summary: Travis CI fails for dependency download Key: ZEPPELIN-4192 URL: https://issues.apache.org/jira/browse/ZEPPELIN-4192 Project: Zeppelin

[GitHub] [zeppelin] r-kamath commented on issue #3379: [ZEPPELIN-4188] Upgrade Jetty to 9.4.18.v20190429

2019-06-13 Thread GitBox
r-kamath commented on issue #3379: [ZEPPELIN-4188] Upgrade Jetty to 9.4.18.v20190429 URL: https://github.com/apache/zeppelin/pull/3379#issuecomment-501560862 @jongyoul ping This is an automated message from the Apache Git

[GitHub] [zeppelin] r-kamath commented on issue #3379: [ZEPPELIN-4188] Upgrade Jetty to 9.4.18.v20190429

2019-06-13 Thread GitBox
r-kamath commented on issue #3379: [ZEPPELIN-4188] Upgrade Jetty to 9.4.18.v20190429 URL: https://github.com/apache/zeppelin/pull/3379#issuecomment-501560744 CI failure is unrelated. Will merge this if there are no more discussions.