[GitHub] [zeppelin] conker84 commented on issue #3485: [ZEPPELIN-3548] KSQL Interpreter for Zeppelin

2019-10-12 Thread GitBox
conker84 commented on issue #3485: [ZEPPELIN-3548] KSQL Interpreter for Zeppelin URL: https://github.com/apache/zeppelin/pull/3485#issuecomment-541374677 @rmoff please look also here This is an automated message from the Apach

[GitHub] [zeppelin] conker84 opened a new pull request #3485: [ZEPPELIN-3548] KSQL Interpreter for Zeppelin

2019-10-12 Thread GitBox
conker84 opened a new pull request #3485: [ZEPPELIN-3548] KSQL Interpreter for Zeppelin URL: https://github.com/apache/zeppelin/pull/3485 ### What is this PR for? This PR adds the support to [KSQL](https://docs.confluent.io/current/ksql/docs/developer-guide/api.html) ### What typ

[GitHub] [zeppelin] elouazzany opened a new pull request #3484: [ZEPPELIN-4368][Docker image] run zeppelin server run under Non-root user

2019-10-12 Thread GitBox
elouazzany opened a new pull request #3484: [ZEPPELIN-4368][Docker image] run zeppelin server run under Non-root user URL: https://github.com/apache/zeppelin/pull/3484 ### What is this PR for? this PR add an improvement to zeppelin docker image by running zeppelin server under non-root

[GitHub] [zeppelin] alexott commented on issue #3483: [ZEPPELIN-4377] Interpreter doesn't start if name has space

2019-10-12 Thread GitBox
alexott commented on issue #3483: [ZEPPELIN-4377] Interpreter doesn't start if name has space URL: https://github.com/apache/zeppelin/pull/3483#issuecomment-541326858 I agree - but we still need to make interpreter more robust... I'll create a separate JIRA to fix the UI to allow onl

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

2019-10-12 Thread GitBox
zjffdu 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_r334238033 ## File path: scripts/docker/zeppelin-server/Dockerfile ## @@ -0,0 +1,79 @@ +# Licensed to the

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

2019-10-12 Thread GitBox
zjffdu 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_r334238070 ## File path: scripts/docker/zeppelin-server/Dockerfile ## @@ -0,0 +1,79 @@ +# Licensed to the

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

2019-10-12 Thread GitBox
zjffdu 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_r334238016 ## File path: scripts/docker/zeppelin-interpreter/Dockerfile_base ## @@ -0,0 +1,55 @@ +# Licen

[GitHub] [zeppelin] zjffdu commented on issue #3483: [ZEPPELIN-4377] Interpreter doesn't start if name has space

2019-10-12 Thread GitBox
zjffdu commented on issue #3483: [ZEPPELIN-4377] Interpreter doesn't start if name has space URL: https://github.com/apache/zeppelin/pull/3483#issuecomment-541325753 I don't think we should allow space in interpreter name, frontend would be able recognize such interpreter, such as `%a b`.

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

2019-10-12 Thread GitBox
alexott 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_r334237507 ## File path: scripts/docker/zeppelin-server/Dockerfile ## @@ -0,0 +1,79 @@ +# Licensed to th

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

2019-10-12 Thread GitBox
alexott 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_r334237394 ## File path: scripts/docker/zeppelin-interpreter/Dockerfile_base ## @@ -0,0 +1,55 @@ +# Lice

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

2019-10-12 Thread GitBox
alexott 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_r334237342 ## File path: scripts/docker/zeppelin-interpreter/Dockerfile_base ## @@ -0,0 +1,55 @@ +# Lice

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

2019-10-12 Thread GitBox
alexott 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_r334237548 ## File path: scripts/docker/zeppelin-server/Dockerfile ## @@ -0,0 +1,79 @@ +# Licensed to th

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

2019-10-12 Thread GitBox
alexott 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_r334237415 ## File path: scripts/docker/zeppelin-server/Dockerfile ## @@ -0,0 +1,79 @@ +# Licensed to th

[GitHub] [zeppelin] alexott opened a new pull request #3483: [ZEPPELIN-4377] Interpreter doesn't start if name has space

2019-10-12 Thread GitBox
alexott opened a new pull request #3483: [ZEPPELIN-4377] Interpreter doesn't start if name has space URL: https://github.com/apache/zeppelin/pull/3483 ### What is this PR for? We can create new interpreters based on existing ones, but we don't enforce its name, so it's possible t

[GitHub] [zeppelin] alexott opened a new pull request #3482: ZEPPELIN-4377 (0.8) Interpreter doesn't start if its name has space

2019-10-12 Thread GitBox
alexott opened a new pull request #3482: ZEPPELIN-4377 (0.8) Interpreter doesn't start if its name has space URL: https://github.com/apache/zeppelin/pull/3482 ### What is this PR for? We can create new interpreters based on existing ones, but we don't enforce its name, so it's po

[jira] [Created] (ZEPPELIN-4377) Interpreter doesn't start if interpreter ID contains space

2019-10-12 Thread Alex Ott (Jira)
Alex Ott created ZEPPELIN-4377: -- Summary: Interpreter doesn't start if interpreter ID contains space Key: ZEPPELIN-4377 URL: https://issues.apache.org/jira/browse/ZEPPELIN-4377 Project: Zeppelin

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

2019-10-12 Thread GitBox
elouazzany 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_r334233321 ## File path: scripts/docker/zeppelin-interpreter/Dockerfile_base ## @@ -0,0 +1,53 @@ +# L

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

2019-10-12 Thread GitBox
elouazzany 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_r334233475 ## File path: scripts/docker/zeppelin-interpreter/Dockerfile_base ## @@ -0,0 +1,55 @@ +# L

[GitHub] [zeppelin] curlsli commented on a change in pull request #3425: [ZEPPELIN-4301] js double precision loss problem

2019-10-12 Thread GitBox
curlsli commented on a change in pull request #3425: [ZEPPELIN-4301] js double precision loss problem URL: https://github.com/apache/zeppelin/pull/3425#discussion_r334227172 ## File path: zeppelin-web/src/app/tabledata/tabledata.js ## @@ -60,7 +60,7 @@ export default clas