[GitHub] [zeppelin] zjffdu commented on issue #3462: ZEPPELIN-4138 Zeppelin Notebooks in vue.js framework

2019-10-08 Thread GitBox
zjffdu commented on issue #3462: ZEPPELIN-4138 Zeppelin Notebooks in vue.js framework URL: https://github.com/apache/zeppelin/pull/3462#issuecomment-539848627 @malayhm I have merged this PR into branch web_vue, could you close it ? --

[GitHub] [zeppelin] asfgit closed pull request #3460: [WIP] [ZEPPELIN-4350]. Paragraph pending for a long time when interpreter process fails to launch

2019-10-08 Thread GitBox
asfgit closed pull request #3460: [WIP] [ZEPPELIN-4350]. Paragraph pending for a long time when interpreter process fails to launch URL: https://github.com/apache/zeppelin/pull/3460 This is an automated message from the Apac

[GitHub] [zeppelin] asfgit closed pull request #3477: [ZEPPELIN-4367]: Streaming tables are not working anymore

2019-10-08 Thread GitBox
asfgit closed pull request #3477: [ZEPPELIN-4367]: Streaming tables are not working anymore URL: https://github.com/apache/zeppelin/pull/3477 This is an automated message from the Apache Git Service. To respond to the messag

[GitHub] [zeppelin] zjffdu opened a new pull request #3478: [ZEPPELIN-4369] Redundant line separator for multiple text output

2019-10-08 Thread GitBox
zjffdu opened a new pull request #3478: [ZEPPELIN-4369] Redundant line separator for multiple text output URL: https://github.com/apache/zeppelin/pull/3478 ### What is this PR for? There's redundant line separator when there's multiple text output. The root cause is that each `%text

[GitHub] [zeppelin] zjffdu commented on issue #3471: [ZEPPELIN-4362] Remove 'flink' from build matrix from travis.yml and some minor improvement

2019-10-08 Thread GitBox
zjffdu commented on issue #3471: [ZEPPELIN-4362] Remove 'flink' from build matrix from travis.yml and some minor improvement URL: https://github.com/apache/zeppelin/pull/3471#issuecomment-539796453 LGTM This is an automated m

[GitHub] [zeppelin] zjffdu commented on issue #3430: [ZEPPELIN-4312] enable livy to run as remote interpreter in docker

2019-10-08 Thread GitBox
zjffdu commented on issue #3430: [ZEPPELIN-4312] enable livy to run as remote interpreter in docker URL: https://github.com/apache/zeppelin/pull/3430#issuecomment-539783294 Thanks @zlosim will merge if no more comments This

[GitHub] [zeppelin] asfgit closed pull request #3472: [ZEPPELIN-4364]credentials.json should use Hadoop Config Storage

2019-10-08 Thread GitBox
asfgit closed pull request #3472: [ZEPPELIN-4364]credentials.json should use Hadoop Config Storage URL: https://github.com/apache/zeppelin/pull/3472 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [zeppelin] zjffdu commented on issue #3473: [ZEPPELIN-4360] Always use Scala 2.11 for Ignite

2019-10-08 Thread GitBox
zjffdu commented on issue #3473: [ZEPPELIN-4360] Always use Scala 2.11 for Ignite URL: https://github.com/apache/zeppelin/pull/3473#issuecomment-539781994 @alexott There's some conflict after I merge #3474 , could you help fix that ? Thanks

[GitHub] [zeppelin] asfgit closed pull request #3474: [ZEPPELIN-4359] Interpreter list is incorrect after recent changes

2019-10-08 Thread GitBox
asfgit closed pull request #3474: [ZEPPELIN-4359] Interpreter list is incorrect after recent changes URL: https://github.com/apache/zeppelin/pull/3474 This is an automated message from the Apache Git Service. To respond to t

[GitHub] [zeppelin] zjffdu commented on issue #3475: [ZEPPELIN-4353] Move flink specific logic into FlinkInterpreterLauncher

2019-10-08 Thread GitBox
zjffdu commented on issue #3475: [ZEPPELIN-4353] Move flink specific logic into FlinkInterpreterLauncher URL: https://github.com/apache/zeppelin/pull/3475#issuecomment-539780632 Will merge if no more comments This is an autom

[GitHub] [zeppelin] zjffdu commented on issue #3477: [ZEPPELIN-4367]: Streaming tables are not working anymore

2019-10-08 Thread GitBox
zjffdu commented on issue #3477: [ZEPPELIN-4367]: Streaming tables are not working anymore URL: https://github.com/apache/zeppelin/pull/3477#issuecomment-539777634 Thanks @conker84 LGTM, will merge if no more comments This is

[GitHub] [zeppelin] zjffdu commented on issue #3466: [Zeppelin-4341] fix role setting when using multiple realms

2019-10-08 Thread GitBox
zjffdu commented on issue #3466: [Zeppelin-4341] fix role setting when using multiple realms URL: https://github.com/apache/zeppelin/pull/3466#issuecomment-539775118 Thanks @primerano LGTM This is an automated message from th

[jira] [Created] (ZEPPELIN-4371) Docker container 0.8.2 on mac os catalina does not work

2019-10-08 Thread Ramanujam Jagannath (Jira)
Ramanujam Jagannath created ZEPPELIN-4371: - Summary: Docker container 0.8.2 on mac os catalina does not work Key: ZEPPELIN-4371 URL: https://issues.apache.org/jira/browse/ZEPPELIN-4371 Project:

[GitHub] [zeppelin] primerano commented on a change in pull request #3466: [Zeppelin-4341] fix role setting when using multiple realms

2019-10-08 Thread GitBox
primerano commented on a change in pull request #3466: [Zeppelin-4341] fix role setting when using multiple realms URL: https://github.com/apache/zeppelin/pull/3466#discussion_r332694912 ## File path: zeppelin-server/src/main/java/org/apache/zeppelin/realm/LdapRealm.java #

[jira] [Created] (ZEPPELIN-4370) Problem accessing /

2019-10-08 Thread Ramanujam Jagannath (Jira)
Ramanujam Jagannath created ZEPPELIN-4370: - Summary: Problem accessing / Key: ZEPPELIN-4370 URL: https://issues.apache.org/jira/browse/ZEPPELIN-4370 Project: Zeppelin Issue Type: Bug

[GitHub] [zeppelin] conker84 commented on issue #3477: [ZEPPELIN-4367]: Streaming tables are not working anymore

2019-10-08 Thread GitBox
conker84 commented on issue #3477: [ZEPPELIN-4367]: Streaming tables are not working anymore URL: https://github.com/apache/zeppelin/pull/3477#issuecomment-539627358 The video also shows that the changes provided by [ZEPPELIN-3690] (#3116) are still valid :) --

Re: KSQL interpreter

2019-10-08 Thread Andrea Santurbano
I had the time to check into the code. I fixed the bug please look into this: https://github.com/apache/zeppelin/pull/3477 Il giorno mar 8 ott 2019 alle ore 15:25 Jeff Zhang ha scritto: > Thanks for reporting this issue. > > Andrea Santurbano 于2019年10月8日周二 下午6:55写道: > > > It's a bug in the visu

[GitHub] [zeppelin] conker84 opened a new pull request #3477: [ZEPPELIN-4367]: Streaming tables are not working anymore

2019-10-08 Thread GitBox
conker84 opened a new pull request #3477: [ZEPPELIN-4367]: Streaming tables are not working anymore URL: https://github.com/apache/zeppelin/pull/3477 ### What is this PR for? The streaming table support was broken due to [ZEPPELIN-3690] (#3116) fix PR fixes the bug ### What

[GitHub] [zeppelin] Tagar commented on issue #3462: ZEPPELIN-4138 Zeppelin Notebooks in vue.js framework

2019-10-08 Thread GitBox
Tagar commented on issue #3462: ZEPPELIN-4138 Zeppelin Notebooks in vue.js framework URL: https://github.com/apache/zeppelin/pull/3462#issuecomment-539622676 @zjffdu Thank you for pointing that there is a separate repo for Zeppelin with latest version of Angular - I didn't know that. Would

[GitHub] [zeppelin] primerano commented on a change in pull request #3466: [Zeppelin-4341] fix role setting when using multiple realms

2019-10-08 Thread GitBox
primerano commented on a change in pull request #3466: [Zeppelin-4341] fix role setting when using multiple realms URL: https://github.com/apache/zeppelin/pull/3466#discussion_r332641216 ## File path: zeppelin-server/src/main/java/org/apache/zeppelin/realm/LdapRealm.java #

[GitHub] [zeppelin] primerano opened a new pull request #3476: [ZEPPELIN-4341] update getRoles to handle any upstream exception

2019-10-08 Thread GitBox
primerano opened a new pull request #3476: [ZEPPELIN-4341] update getRoles to handle any upstream exception URL: https://github.com/apache/zeppelin/pull/3476 See https://github.com/apache/zeppelin/pull/3466 for reasons. 3466 was for 0.8 and this PR is for master. Only once file needed

[jira] [Created] (ZEPPELIN-4369) Redundant line separator for multiple text output

2019-10-08 Thread Jeff Zhang (Jira)
Jeff Zhang created ZEPPELIN-4369: Summary: Redundant line separator for multiple text output Key: ZEPPELIN-4369 URL: https://issues.apache.org/jira/browse/ZEPPELIN-4369 Project: Zeppelin Iss

Re: KSQL interpreter

2019-10-08 Thread Jeff Zhang
Thanks for reporting this issue. Andrea Santurbano 于2019年10月8日周二 下午6:55写道: > It's a bug in the visualization (I guess), also the example provided here: > > https://github.com/apache/zeppelin/pull/2809 > > is not working anymore > > I opened this issue: > > https://issues.apache.org/jira/browse/Z

[jira] [Created] (ZEPPELIN-4368) [Docker image] for security reason the zeppelin server should run under Non-root user

2019-10-08 Thread Abdelouahab EL OUAZZANY (Jira)
Abdelouahab EL OUAZZANY created ZEPPELIN-4368: - Summary: [Docker image] for security reason the zeppelin server should run under Non-root user Key: ZEPPELIN-4368 URL: https://issues.apache.org/jira/bro

[GitHub] [zeppelin] primerano commented on a change in pull request #3466: [Zeppelin-4341] fix role setting when using multiple realms

2019-10-08 Thread GitBox
primerano commented on a change in pull request #3466: [Zeppelin-4341] fix role setting when using multiple realms URL: https://github.com/apache/zeppelin/pull/3466#discussion_r332452885 ## File path: zeppelin-server/src/main/java/org/apache/zeppelin/utils/SecurityUtils.java #

Re: KSQL interpreter

2019-10-08 Thread Andrea Santurbano
It's a bug in the visualization (I guess), also the example provided here: https://github.com/apache/zeppelin/pull/2809 is not working anymore I opened this issue: https://issues.apache.org/jira/browse/ZEPPELIN-4367 If someone can mentor me on how to fix this, I can do it. Thanks a lot Andrea

[jira] [Created] (ZEPPELIN-4367) Streaming tables are not working anymore

2019-10-08 Thread Andrea Santurbano (Jira)
Andrea Santurbano created ZEPPELIN-4367: --- Summary: Streaming tables are not working anymore Key: ZEPPELIN-4367 URL: https://issues.apache.org/jira/browse/ZEPPELIN-4367 Project: Zeppelin

Re: KSQL interpreter

2019-10-08 Thread Andrea Santurbano
Following the Interpreter log (seems to be good): https://gist.github.com/conker84/aad9b9a220e781b30d5aab1ddc74a4d7#file-zeppelin-interpreter-debug-log Il giorno mar 8 ott 2019 alle ore 11:38 Andrea Santurbano ha scritto: > Seems to be different, please check here: > > https://gist.github.com/co

[GitHub] [zeppelin] zjffdu edited a comment on issue #3462: ZEPPELIN-4138 Zeppelin Notebooks in vue.js framework

2019-10-08 Thread GitBox
zjffdu edited a comment on issue #3462: ZEPPELIN-4138 Zeppelin Notebooks in vue.js framework URL: https://github.com/apache/zeppelin/pull/3462#issuecomment-539440284 > Thank you @malayhm and @ieglonew01f > > IMHO it would actually be great to see VueJS to be part of mainstream Zeppe

[GitHub] [zeppelin] zjffdu edited a comment on issue #3462: ZEPPELIN-4138 Zeppelin Notebooks in vue.js framework

2019-10-08 Thread GitBox
zjffdu edited a comment on issue #3462: ZEPPELIN-4138 Zeppelin Notebooks in vue.js framework URL: https://github.com/apache/zeppelin/pull/3462#issuecomment-539440284 > Thank you @malayhm and @ieglonew01f > > IMHO it would actually be great to see VueJS to be part of mainstream Zeppe

[GitHub] [zeppelin] zjffdu commented on issue #3462: ZEPPELIN-4138 Zeppelin Notebooks in vue.js framework

2019-10-08 Thread GitBox
zjffdu commented on issue #3462: ZEPPELIN-4138 Zeppelin Notebooks in vue.js framework URL: https://github.com/apache/zeppelin/pull/3462#issuecomment-539440284 > Thank you @malayhm and @ieglonew01f > > IMHO it would actually be great to see VueJS to be part of mainstream Zeppelin.

Re: KSQL interpreter

2019-10-08 Thread Andrea Santurbano
Seems to be different, please check here: https://gist.github.com/conker84/4a54df4c73dab310cdc9284f15e90075 Can you mentor me about what I'm wrong? Il giorno mar 8 ott 2019 alle ore 11:13 Jeff Zhang ha scritto: > The the following line is suspicious to me. > > https://github.com/conker84/zep

Re: KSQL interpreter

2019-10-08 Thread Jeff Zhang
The the following line is suspicious to me. https://github.com/conker84/zeppelin/blob/kafka/kafka/src/main/java/org/apache/zeppelin/kafka/KafkaKSQLInterpreter.java#L120 Could you add the following line to log4j.properties so that you can see more logging info. log4j.logger.org.apache.zeppelin

Re: KSQL interpreter

2019-10-08 Thread Andrea Santurbano
Did you check the animated gif that I provided in the first email? Looks like a problem in the visualization. Btw I expect the same behaviour as here: https://github.com/apache/zeppelin/pull/2809 and https://github.com/apache/zeppelin/pull/2873 Il giorno mar 8 ott 2019 alle ore 10:04 Jeff Zhan

[GitHub] [zeppelin] zjffdu commented on issue #3472: [ZEPPELIN-4364]credentials.json should use Hadoop Config Storage

2019-10-08 Thread GitBox
zjffdu commented on issue #3472: [ZEPPELIN-4364]credentials.json should use Hadoop Config Storage URL: https://github.com/apache/zeppelin/pull/3472#issuecomment-539403091 LGTM, will merge if no more comments This is an automa

[GitHub] [zeppelin] codenamelxl commented on issue #3472: [ZEPPELIN-4364]credentials.json should use Hadoop Config Storage

2019-10-08 Thread GitBox
codenamelxl commented on issue #3472: [ZEPPELIN-4364]credentials.json should use Hadoop Config Storage URL: https://github.com/apache/zeppelin/pull/3472#issuecomment-539402534 Hi @zjffdu, I have updated the PR description. Please suggest if I can make it better. Thanks. --

Re: KSQL interpreter

2019-10-08 Thread Jeff Zhang
I encounter some issues when integrating flink streaming, not sure the same as you. Just curious to know what is your expected output if it is streaming job and output is generated continuously. Andrea Santurbano 于2019年10月8日周二 下午4:01写道: > No clue at all? > > Il giorno lun 7 ott 2019 alle ore 14

Re: KSQL interpreter

2019-10-08 Thread Andrea Santurbano
No clue at all? Il giorno lun 7 ott 2019 alle ore 14:14 Andrea Santurbano ha scritto: > Could it be a bug in the front-end? Or Am I missing something? > > Il giorno lun 7 ott 2019 alle ore 09:39 Andrea Santurbano < > sant...@gmail.com> ha scritto: > >> More or less yes (it transforms sql stateme

[GitHub] [zeppelin] zjffdu commented on issue #3472: [ZEPPELIN-4364]credentials.json should use Hadoop Config Storage

2019-10-08 Thread GitBox
zjffdu commented on issue #3472: [ZEPPELIN-4364]credentials.json should use Hadoop Config Storage URL: https://github.com/apache/zeppelin/pull/3472#issuecomment-539383480 Thanks for the contribution @codenamelxl , could you update your PR description ?

[GitHub] [zeppelin] zjffdu commented on a change in pull request #3466: [Zeppelin-4341] fix role setting when using multiple realms

2019-10-08 Thread GitBox
zjffdu commented on a change in pull request #3466: [Zeppelin-4341] fix role setting when using multiple realms URL: https://github.com/apache/zeppelin/pull/3466#discussion_r332367273 ## File path: zeppelin-server/src/main/java/org/apache/zeppelin/utils/SecurityUtils.java

[GitHub] [zeppelin] zjffdu commented on a change in pull request #3466: [Zeppelin-4341] fix role setting when using multiple realms

2019-10-08 Thread GitBox
zjffdu commented on a change in pull request #3466: [Zeppelin-4341] fix role setting when using multiple realms URL: https://github.com/apache/zeppelin/pull/3466#discussion_r332367295 ## File path: zeppelin-server/src/main/java/org/apache/zeppelin/realm/LdapRealm.java ##

[GitHub] [zeppelin] zjffdu commented on issue #3460: [WIP] [ZEPPELIN-4350]. Paragraph pending for a long time when interpreter process fails to launch

2019-10-08 Thread GitBox
zjffdu commented on issue #3460: [WIP] [ZEPPELIN-4350]. Paragraph pending for a long time when interpreter process fails to launch URL: https://github.com/apache/zeppelin/pull/3460#issuecomment-539381787 Will merge if no more comments ---

[GitHub] [zeppelin] zjffdu opened a new pull request #3475: [ZEPPELIN-4353] Move flink specific logic into FlinkInterpreterLauncher

2019-10-08 Thread GitBox
zjffdu opened a new pull request #3475: [ZEPPELIN-4353] Move flink specific logic into FlinkInterpreterLauncher URL: https://github.com/apache/zeppelin/pull/3475 ### What is this PR for? This PR is a refactoring PR which introduce a new plugin module `launcher/flink` for launching f