[GitHub] zeppelin issue #2163: [ZEPPELIN-2287] Add more test to ensure 'RunOnSelectio...

2017-03-24 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2163 @Leemoonsoo Right it should be passed without refreshing. So I removed that part and added `waitForParagraph(1, "FINISHED");` before every selections for the stability. Now CI is green now.

[GitHub] zeppelin pull request #2174: [ZEPELIN-2209] Add pagination to Helium menu

2017-03-24 Thread AhyoungRyu
GitHub user AhyoungRyu reopened a pull request: https://github.com/apache/zeppelin/pull/2174 [ZEPELIN-2209] Add pagination to Helium menu ### What is this PR for? Like I did in #2126, added pagination to Helium menu in Zeppelin. 10 pkgs will be shown per 1 page.

[GitHub] zeppelin pull request #2174: [ZEPELIN-2209] Add pagination to Helium menu

2017-03-24 Thread AhyoungRyu
Github user AhyoungRyu closed the pull request at: https://github.com/apache/zeppelin/pull/2174 --- 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 #2180: [gh-pages][ZEPPELIN-2303] Fix lately published ...

2017-03-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2180 --- 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 pull request #2184: [MINOR][ZEPPELIN-2306] Improve alignment of com...

2017-03-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2184 --- 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 pull request #2185: [MINOR][ZEPPELIN-2308] Fix vis icon button grou...

2017-03-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2185 --- 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 #2186: [ZEPPELIN-2229] revert ndv3 version from 1.8.5 to 1.7....

2017-03-24 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2186 @soralee Thanks for the explanation. I tested this branch to make sure it works well or not, but when I select "Line chart" below error msg shows up. ```

[GitHub] zeppelin pull request #2168: [ZEPPELIN-1889] [FIX] Multiple groups in chart ...

2017-03-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2168 --- 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 pull request #2173: [ZEPPELIN-2291] Notebook commit dropdown cut-of...

2017-03-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2173 --- 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

[jira] [Created] (ZEPPELIN-2316) Spark-Interpreter Dependencies not correctly declared

2017-03-24 Thread Daniel Langerenken (JIRA)
Daniel Langerenken created ZEPPELIN-2316: Summary: Spark-Interpreter Dependencies not correctly declared Key: ZEPPELIN-2316 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2316 Project:

[GitHub] zeppelin issue #2113: [ZEPPELIN-2190] Support custom web development port

2017-03-24 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/2113 @1ambda Hi, I just updated as your comments. Could you check this again? Thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] zeppelin issue #2187: [ZEPPELIN-2314] Fix watcher websocket origin header

2017-03-24 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/2187 /cc @anthonycorbacho @cloverhearts @Leemoonsoo plz help review this one --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] zeppelin pull request #2188: [MINOR] Job parameters synchronization

2017-03-24 Thread AlexanderShoshin
GitHub user AlexanderShoshin opened a pull request: https://github.com/apache/zeppelin/pull/2188 [MINOR] Job parameters synchronization ### What is this PR for? We need to synchronize setting up job result, exception, errorMessage and dateFinished in the end of `Job.run()`

[GitHub] zeppelin issue #1976: [ZEPPELIN-2037] Restart button does not work

2017-03-24 Thread del680202
Github user del680202 commented on the issue: https://github.com/apache/zeppelin/pull/1976 @zjffdu @AhyoungRyu Yes, I tested it by latest branch-0.7. Latest commit ``` commit 53e90578621574f466a58ad02a6577bca6335a5a Author: Guillermo Cabrera

[GitHub] zeppelin issue #2135: Groovy Interpreter for Apazhe Zeppelin [ZEPPELIN-2176]

2017-03-24 Thread dlukyanov
Github user dlukyanov commented on the issue: https://github.com/apache/zeppelin/pull/2135 @Leemoonsoo Interesting) I did not knew that... i'll take a look how it's implemented... With doc - i'll move it --- If your project is set up for it, you can reply to this email and have

[GitHub] zeppelin pull request #2187: [ZEPPELIN-2314] Fix watcher websocket origin he...

2017-03-24 Thread khalidhuseynov
GitHub user khalidhuseynov opened a pull request: https://github.com/apache/zeppelin/pull/2187 [ZEPPELIN-2314] Fix watcher websocket origin header ### What is this PR for? this is to add `Origin` header to ws client, and fix ws connection error after merging ZEPPELIN-2288,

[jira] [Created] (ZEPPELIN-2314) Watcher websocket connection origin null error

2017-03-24 Thread Khalid Huseynov (JIRA)
Khalid Huseynov created ZEPPELIN-2314: - Summary: Watcher websocket connection origin null error Key: ZEPPELIN-2314 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2314 Project: Zeppelin

[GitHub] zeppelin issue #1976: [ZEPPELIN-2037] Restart button does not work

2017-03-24 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1976 @del680202 I can not reproduce the issue on 0.7.1 rc2, regarding your instruction, when you do step 2, zeppelin would ask you to restart the interpreter, did you click the button to restart it ?

[GitHub] zeppelin issue #1976: [ZEPPELIN-2037] Restart button does not work

2017-03-24 Thread del680202
Github user del680202 commented on the issue: https://github.com/apache/zeppelin/pull/1976 @AhyoungRyu Thanks for your reply. >>But I seem when I edit interpreter config then the zombie process will be occurred Sorry, I miss this test case. I test it by current

Re: [VOTE] Release Apache Zeppelin 0.7.1 (RC2)

2017-03-24 Thread Jeff Zhang
Sorry, I need to -1 again, as I just found that ZEPPELIN-1984 is also needed to backport. Otherwise no error message displayed in frontend when interpreter fail to start. Sora Lee 于2017年3月24日周五 下午2:14写道: > I faced a error when switching the scatter chart, the error message

[GitHub] zeppelin issue #2180: [gh-pages][ZEPPELIN-2303] Fix lately published package...

2017-03-24 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2180 CI failure is not related with this change since it's based on `gh-pages` branch. Merge into `gh-pages` if there are no more comments. --- If your project is set up for it, you can

[GitHub] zeppelin issue #2186: [ZEPPELIN-2229] revert ndv3 version from 1.8.5 to 1.7....

2017-03-24 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/2186 @AhyoungRyu @Leemoonsoo I just checked it! We can either revert or merge it. the part I didn't include don't matter. --- If your project is set up for it, you can reply to this email and have

[GitHub] zeppelin issue #1976: [ZEPPELIN-2037] Restart button does not work

2017-03-24 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1976 @del680202 Thanks for your time and effort to test it out. >But I seem when I edit interpreter config then the zombie process will be occurred I checked it with current

[GitHub] zeppelin pull request #1872: [ZEPPELIN-1922] Exclude jackson-core and jackso...

2017-03-24 Thread epahomov
Github user epahomov closed the pull request at: https://github.com/apache/zeppelin/pull/1872 --- 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 issue #2134: [ZEPPELIN-2257] notification about incompleteness of d...

2017-03-24 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2134 @tinkoff-dwh Thanks! Looks good :) --- 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 issue #2186: [ZEPPELIN-2229] revert ndv3 version from 1.8.5 to 1.7....

2017-03-24 Thread soralee
Github user soralee commented on the issue: https://github.com/apache/zeppelin/pull/2186 @AhyoungRyu I did! Thanks for reproduce! Let me check this part I didn't include. If it is possible, we had better revert it. --- If your project is set up for it, you can reply to this

Re: [VOTE] Release Apache Zeppelin 0.7.1 (RC2)

2017-03-24 Thread Sora Lee
I faced a error when switching the scatter chart, the error message that is "cannot property 'x' of null" appear in web development tool of web browser. so I created this PR (https://github.com/apache/zeppelin/pull/2186 ). Shell we include this patch in 0.7.1 release? Thanks, Sora On Fri, Mar

[GitHub] zeppelin issue #2186: [ZEPPELIN-2229] revert ndv3 version from 1.8.5 to 1.7....

2017-03-24 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2186 @soralee Yeah I can see the error when we have large data. ![again](https://cloud.githubusercontent.com/assets/10060731/24282230/8307f99a-10a1-11e7-964e-d3b57be7943a.gif) I tested

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

2017-03-24 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2134 > I can add the close button @tinkoff-dwh Yeah that's better. Having close button will be better for other cases as well. --- If your project is set up for it, you can reply to