[GitHub] zeppelin pull request #2153: [ZEPPELIN-2282] NPE on NotebookServer.java when...

2017-03-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2153 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] zeppelin issue #2111: [ZEPPELIN-2230] clear bower_contents if bower.json cha...

2017-03-17 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2111 @Remilito I've reverted this PR because it breaks all CI build. ``` $ changedfiles=$(git diff --name-only $TRAVIS_COMMIT_RANGE) fatal: ambiguous argument '38d0207cd99d...362b7ca3c209':

[GitHub] zeppelin pull request #2149: [HOTFIX][ZEPPELIN-2037][ZEPPELIN-1832] Restart ...

2017-03-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2149 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] zeppelin issue #2149: [HOTFIX][ZEPPELIN-2037][ZEPPELIN-1832] Restart with se...

2017-03-17 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2149 Thanks for the review. Will merge it --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] zeppelin pull request #2153: [ZEPPELIN-2282] NPE on NotebookServer.java when...

2017-03-17 Thread Leemoonsoo
GitHub user Leemoonsoo opened a pull request: https://github.com/apache/zeppelin/pull/2153 [ZEPPELIN-2282] NPE on NotebookServer.java when client send null ticket. ### What is this PR for? When client (zeppelin-web) send message with null ticket, NotebookServer does not handle

[jira] [Created] (ZEPPELIN-2282) NPE on NotebookServer.java[onMessage]:356

2017-03-17 Thread Lee moon soo (JIRA)
Lee moon soo created ZEPPELIN-2282: -- Summary: NPE on NotebookServer.java[onMessage]:356 Key: ZEPPELIN-2282 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2282 Project: Zeppelin Issue

[GitHub] zeppelin issue #2143: [ZEPPELIN-2266] Parse helium description as HTML

2017-03-17 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2143 LGTM and merge to master if no further comments! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] zeppelin issue #2134: [ZEPPELIN-2257] notification about incompleteness of d...

2017-03-17 Thread tinkoff-dwh
Github user tinkoff-dwh commented on the issue: https://github.com/apache/zeppelin/pull/2134 @Leemoonsoo *location* I think it's better to display above table because the output can be large and user will not see message. *style* Bootstreap attracts attention and

[GitHub] zeppelin issue #2144: [ZEPPELIN-2267] Improve Helium Enable / Disable Dialog...

2017-03-17 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2144 Cool improvement! LGTM and merge master if no further discussions. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] zeppelin issue #2134: [ZEPPELIN-2257] notification about incompleteness of d...

2017-03-17 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2134 Message location Because of existing message `Results are limited by X.` and `Output exceeds 10240. Truncated.` are displayed below the table. I think showing `Attention!

[GitHub] zeppelin pull request #2152: [MINOR] Display chart by default in basic tutor...

2017-03-17 Thread Leemoonsoo
GitHub user Leemoonsoo opened a pull request: https://github.com/apache/zeppelin/pull/2152 [MINOR] Display chart by default in basic tutorial notebook ### What is this PR for? From some point, "Zeppelin Tutorial/Basic Features (Spark)" tutorial notebook does not display chart

[GitHub] zeppelin issue #2134: [ZEPPELIN-2257] notification about incompleteness of d...

2017-03-17 Thread tinkoff-dwh
Github user tinkoff-dwh commented on the issue: https://github.com/apache/zeppelin/pull/2134 @Leemoonsoo what final version, bootstrap? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] zeppelin issue #2131: ZEPPELIN-2256. poll job progress is called twice

2017-03-17 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2131 Merge to master and branch-0.7 if no further discussions --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does

[GitHub] zeppelin issue #2134: [ZEPPELIN-2257] notification about incompleteness of d...

2017-03-17 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2134 In addition to @prabhjyotsingh mentions, there are another type of message controlled by `ZEPPELIN_INTERPRETER_OUTPUT_LIMIT` env variable.

[GitHub] zeppelin issue #2099: [ZEPPELIN-2106] providing paragraph config in create n...

2017-03-17 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2099 @Remilito Thanks for explain. Let's say `graph: {mode: {...}}` is api A, and `config: {results: [{graph: mode: {...}}]}` is api B. When user use api A with multiple results,

[jira] [Created] (ZEPPELIN-2281) If execution of paragrah is aborted then the previous results are cleared

2017-03-17 Thread Sachin Janani (JIRA)
Sachin Janani created ZEPPELIN-2281: --- Summary: If execution of paragrah is aborted then the previous results are cleared Key: ZEPPELIN-2281 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2281

[GitHub] zeppelin issue #2149: [HOTFIX][ZEPPELIN-2037][ZEPPELIN-1832] Restart with se...

2017-03-17 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2149 Tested Per Note, Scoped Per Note, Isolated Per User, Scoped Per User, Isolated Per User, Scoped + Per Note, Scoped Per User, Isolated + Per Note, Scoped

Re: [CI] second job exceeds travis.org max job time

2017-03-17 Thread Remy Gayet
nevermind. One of my test is hanging because of a bug somewhere in my code. On Fri, Mar 17, 2017 at 12:15 PM, Remy Gayet wrote: > Hello guys, > > the second build configuration (env below) is now taking too much time ( > > 49min) and gets killed, as per travis.org policy (

[GitHub] zeppelin issue #2093: [ZEPPELIN-2277] Env variable to configure maven centra...

2017-03-17 Thread andreaTP
Github user andreaTP commented on the issue: https://github.com/apache/zeppelin/pull/2093 Thanks a lot! appreciated this welcoming environment! (I was aiming to become committer number 200 also ) --- If your project is set up for it, you can reply to this email and have your reply

[jira] [Created] (ZEPPELIN-2280) Spark Default Interpreter Setting Changed in interpreter.json not Reflected in UI

2017-03-17 Thread Alex Liu (JIRA)
Alex Liu created ZEPPELIN-2280: -- Summary: Spark Default Interpreter Setting Changed in interpreter.json not Reflected in UI Key: ZEPPELIN-2280 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2280

[GitHub] zeppelin issue #2098: [ZEPPELIN-2217] AdvancedTransformation for Visualizati...

2017-03-17 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2098 @1ambda Great work! Let me test this interesting feature in this weekend. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] zeppelin pull request #2151: [ZEPPELIN-465] Be able to run paragraph and the...

2017-03-17 Thread Remilito
GitHub user Remilito opened a pull request: https://github.com/apache/zeppelin/pull/2151 [ZEPPELIN-465] Be able to run paragraph and the following ones ### What is this PR for? Allow to run a paragraph **and** the ones following. This is useful for instance if you are

[GitHub] zeppelin pull request #2093: [ZEPPELIN-2277] Env variable to configure maven...

2017-03-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2093 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] zeppelin issue #2150: [ZEPPELIN-2278] Env variable to configure Npm registry

2017-03-17 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2150 It might be due to flaky tests. If so, please just restart failed jobs. I'v just looked your travis build and it's ok - https://travis-ci.org/andreaTP/zeppelin/builds/212066501

[GitHub] zeppelin issue #2098: [ZEPPELIN-2217] AdvancedTransformation for Visualizati...

2017-03-17 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2098 I'v just updated the PR description. Please help review this @AhyoungRyu :) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] zeppelin issue #2150: [ZEPPELIN-2278] Env variable to configure Npm registry

2017-03-17 Thread andreaTP
Github user andreaTP commented on the issue: https://github.com/apache/zeppelin/pull/2150 Thanks for the feedback @1ambda , travis should be configured properly now, I will try to dig into but it looks tests spuriously fail, anyhow any help on getting the right direction is highly

[GitHub] zeppelin issue #2150: [ZEPPELIN-2278] Env variable to configure Npm registry

2017-03-17 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2150 Hi @andreaTP. Thanks for contribution. - Zeppelin build system relies on travis. So please setup your travis account and travis project for Zeppelin. -

[GitHub] zeppelin issue #2112: [ZEPPELIN-2228] precode for SparkRInterpreter

2017-03-17 Thread tinkoff-dwh
Github user tinkoff-dwh commented on the issue: https://github.com/apache/zeppelin/pull/2112 CI green, https://travis-ci.org/tinkoff-dwh/zeppelin/builds/208970773 , commit is correct 8323392, but status failed (tried reopen) failed. --- If your project is set up for it, you can

[GitHub] zeppelin pull request #2112: [ZEPPELIN-2228] precode for SparkRInterpreter

2017-03-17 Thread tinkoff-dwh
GitHub user tinkoff-dwh reopened a pull request: https://github.com/apache/zeppelin/pull/2112 [ZEPPELIN-2228] precode for SparkRInterpreter ### What is this PR for? Added parameter zeppelin.pyspark.precode to SparkRInterpreter. This is snippet of code which executes when

[GitHub] zeppelin pull request #2112: [ZEPPELIN-2228] precode for SparkRInterpreter

2017-03-17 Thread tinkoff-dwh
Github user tinkoff-dwh closed the pull request at: https://github.com/apache/zeppelin/pull/2112 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[jira] [Created] (ZEPPELIN-2279) Incorrect parsing queries if there are comments

2017-03-17 Thread Tinkoff DWH (JIRA)
Tinkoff DWH created ZEPPELIN-2279: - Summary: Incorrect parsing queries if there are comments Key: ZEPPELIN-2279 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2279 Project: Zeppelin

[GitHub] zeppelin issue #2150: [ZEPPELIN-2278] Env variable to configure Npm registry

2017-03-17 Thread andreaTP
Github user andreaTP commented on the issue: https://github.com/apache/zeppelin/pull/2150 failures looks unrelated ... --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[CI] second job exceeds travis.org max job time

2017-03-17 Thread Remy Gayet
Hello guys, the second build configuration (env below) is now taking too much time ( > 49min) and gets killed, as per travis.org policy ( https://docs.travis-ci.com/user/customizing-the-build#Build-Timeouts). SCALA_VER=\"2.11\" SPARK_VER=\"2.1.0\" HADOOP_VER=\"2.6\" LIVY_VER=\"0.3.0\"

[GitHub] zeppelin issue #2093: [ZEPPELIN-2277] Env variable to configure maven centra...

2017-03-17 Thread andreaTP
Github user andreaTP commented on the issue: https://github.com/apache/zeppelin/pull/2093 updated also commit message here --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] zeppelin pull request #2150: [ZEPPELIN-2278] Env variable to configure Npm r...

2017-03-17 Thread andreaTP
GitHub user andreaTP opened a pull request: https://github.com/apache/zeppelin/pull/2150 [ZEPPELIN-2278] Env variable to configure Npm registry ### What is this PR for? When deploying Zeppelin in private Cloud or within closed networks, it happens that Npm Registry isn't

[jira] [Created] (ZEPPELIN-2278) Env variable to configure Npm registry

2017-03-17 Thread Andrea Peruffo (JIRA)
Andrea Peruffo created ZEPPELIN-2278: Summary: Env variable to configure Npm registry Key: ZEPPELIN-2278 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2278 Project: Zeppelin Issue

[GitHub] zeppelin issue #2141: [ZEPPELIN-1720] Adding tests to verify behaviour of dy...

2017-03-17 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2141 Seems some review comments are mixed up in github side :( I just removed some of them and left comment again. Please let me know if you're confused. --- If your project is set up for

[GitHub] zeppelin issue #2141: [ZEPPELIN-1720] Adding tests to verify behaviour of dy...

2017-03-17 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2141 @guicaro Thanks for your great contribution! I left some comments. Could you take a look at them? Please let me know if there is something that I missed :) --- If your project is set up

[GitHub] zeppelin issue #2093: [ZEPPELIN-2277] Env variable to configure maven centra...

2017-03-17 Thread andreaTP
Github user andreaTP commented on the issue: https://github.com/apache/zeppelin/pull/2093 Yep, opened Jira issue and linked accordingly on PR title. https://issues.apache.org/jira/browse/ZEPPELIN-2277 @Leemoonsoo thanks! --- If your project is set up for it, you can

[jira] [Created] (ZEPPELIN-2277) Env variable to configure maven central repo

2017-03-17 Thread Andrea Peruffo (JIRA)
Andrea Peruffo created ZEPPELIN-2277: Summary: Env variable to configure maven central repo Key: ZEPPELIN-2277 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2277 Project: Zeppelin

[GitHub] zeppelin pull request #2138: [ZEPPELIN-2260] [branch-0.7] Skip node,npm inst...

2017-03-17 Thread Leemoonsoo
Github user Leemoonsoo closed the pull request at: https://github.com/apache/zeppelin/pull/2138 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] zeppelin pull request #2137: [ZEPPELIN-2260] Skip node,npm install and bundl...

2017-03-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2137 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] zeppelin issue #2106: Rewrite PythonInterpreter.

2017-03-17 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2106 @astroshim made [a pullrequest](https://github.com/astroshim/zeppelin/pull/16) to this branch, to bringing docker support back . Please take a look and merge if you think it's okay. --- If