[GitHub] zeppelin pull request #2607: [HOTFIX] fix interpreter.cmd script

2017-10-15 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2607 ---

[GitHub] zeppelin pull request #2625: ZEPPELIN-2988. Zeppelin does not start new Livy...

2017-10-15 Thread zjffdu
GitHub user zjffdu opened a pull request: https://github.com/apache/zeppelin/pull/2625 ZEPPELIN-2988. Zeppelin does not start new Livy session if yarn livy session application is killed ### What is this PR for? This PR will display a warning message when the livy session is

[GitHub] zeppelin issue #2607: [HOTFIX] fix interpreter.cmd script

2017-10-15 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2607 Merge if no more discussion ---

[GitHub] zeppelin issue #2607: [HOTFIX] fix interpreter.cmd script

2017-10-15 Thread tinkoff-dwh
Github user tinkoff-dwh commented on the issue: https://github.com/apache/zeppelin/pull/2607 Ready to review ---

[jira] [Created] (ZEPPELIN-2997) Add configuration for disabling the feature of "auto-restart interpreter on cron execution"

2017-10-15 Thread Keiji Yoshida (JIRA)
Keiji Yoshida created ZEPPELIN-2997: --- Summary: Add configuration for disabling the feature of "auto-restart interpreter on cron execution" Key: ZEPPELIN-2997 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2

[GitHub] zeppelin pull request #2622: ZEPPELIN-2990. Matplotlib sometimes fails in IP...

2017-10-15 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2622 ---

[GitHub] zeppelin issue #2622: ZEPPELIN-2990. Matplotlib sometimes fails in IPythonIn...

2017-10-15 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2622 I will merge it soon as it fix the flaky travis build as well. ---

[GitHub] zeppelin issue #2622: ZEPPELIN-2990. Matplotlib sometimes fails in IPythonIn...

2017-10-15 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2622 @felixcheung Right, this PR is trying to fix these flaky test. ---

[GitHub] zeppelin issue #2624: [ZEPPELIN-2965] Add code completion for livy interpret...

2017-10-15 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2624 IMHO,it is better to have integration test for interpreter. Because interpreter is usually 3rd party library outside zeppelin. Unit test usually has many assumption which may not be true in real

[GitHub] zeppelin issue #2617: ZEPPELIN-2985: correct grammar in keyboard tooltip

2017-10-15 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2617 Thanks @gef756 for the contribution! Merge to master ---

[GitHub] zeppelin issue #2624: [ZEPPELIN-2965] Add code completion for livy interpret...

2017-10-15 Thread pellmont
Github user pellmont commented on the issue: https://github.com/apache/zeppelin/pull/2624 welcome @zjffdu. Actually I am not a big fan of integration tests, because they are usually slow and brittle. In my opinion integration tests should be used sparely, the main functionality sh

[GitHub] zeppelin issue #2624: [ZEPPELIN-2965] Add code completion for livy interpret...

2017-10-15 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2624 Thanks for the contribution @pellmont , but I don't think it make sense to make so large change just for unit test. Actually livy interpreter can run integration which is much better than unit test

[GitHub] zeppelin pull request #2624: [ZEPPELIN-2965] Add code completion for livy in...

2017-10-15 Thread pellmont
GitHub user pellmont opened a pull request: https://github.com/apache/zeppelin/pull/2624 [ZEPPELIN-2965] Add code completion for livy interpreter ### What is this PR for? This PR adds code autocompletion feature to LivyInterpreter. Livy version 0.5 will have an auto completio

[GitHub] zeppelin issue #2601: [ZEPPELIN-2956] [font-end] Downloaded CSV/TSV data wil...

2017-10-15 Thread BruceXu1991
Github user BruceXu1991 commented on the issue: https://github.com/apache/zeppelin/pull/2601 @felixcheung The API(http://ui-grid.info/docs/#/tutorial/206_exporting_data) only support export data as csv format but no tsv format supporting. So if using this API, tsv format related PR s

[GitHub] zeppelin issue #2623: [FIX] fix converter forms to json

2017-10-15 Thread tinkoff-dwh
Github user tinkoff-dwh commented on the issue: https://github.com/apache/zeppelin/pull/2623 @zjffdu @felixcheung done ---