[GitHub] zeppelin issue #2852: [ZEPPELIN-3264] Notebook Snapshot feature.

2018-03-20 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2852 > Git tag regularly is not something feels right [ Usually in practice Git Tag are to be done only once ]. That is my understanding of the snapshot scenario. Snapshot is only readable,

[GitHub] zeppelin issue #2760: ZEPPELIN-3196. Plugin framework for Zeppelin Engine

2018-03-20 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2760 Took a look changes and it looks good to me ---

[GitHub] zeppelin pull request #2803: ZEPPELIN-3232. Allow to write config files to f...

2018-03-20 Thread zjffdu
Github user zjffdu closed the pull request at: https://github.com/apache/zeppelin/pull/2803 ---

[GitHub] zeppelin issue #2760: ZEPPELIN-3196. Plugin framework for Zeppelin Engine

2018-03-20 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2760 @Leemoonsoo @jongyoul @prabhjyotsingh @felixcheung Could you help review it ? Thanks, it is for 0.9. ---

[GitHub] zeppelin issue #2870: [ZEPPELIN-3335] trim property name of interpreter sett...

2018-03-20 Thread rockiee281
Github user rockiee281 commented on the issue: https://github.com/apache/zeppelin/pull/2870 Sorry, I'm not familiar with git rebase ,create PR #2884 instead ---

[GitHub] zeppelin pull request #2884: trim property name of interpreter settings

2018-03-20 Thread rockiee281
GitHub user rockiee281 opened a pull request: https://github.com/apache/zeppelin/pull/2884 trim property name of interpreter settings ### What is this PR for? trim property name of interpreter settings ### What type of PR is it? Improvement ### Todos *

[GitHub] zeppelin issue #2834: [ZEPPELIN-1967] Passing Z variables to Shell and SQL I...

2018-03-20 Thread Tagar
Github user Tagar commented on the issue: https://github.com/apache/zeppelin/pull/2834 @zjffdu, @felixcheung would be exciting to have this feature be part of the 0.8.0 release. Since there is a way to maintain backward compatibility, it seems to be a low hanging fruit as

[GitHub] zeppelin issue #2880: [ZEPPELIN-3351] Fix to build for spark 2.3

2018-03-20 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2880 I think we should try to fix the build profile as in this PR. But we must fix the download source - otherwise given what I’ve seen recently, we cannot release (its not up to me)

[GitHub] zeppelin issue #2852: [ZEPPELIN-3264] Notebook Snapshot feature.

2018-03-20 Thread SleepyThread
Github user SleepyThread commented on the issue: https://github.com/apache/zeppelin/pull/2852 @zjffdu Implementation using Git Tag is also another approach but updating/delete Git tag regularly is not something feels right [ Usually in practice Git Tag are to be done only once ].

[GitHub] zeppelin pull request #2883: ZEPPELIN-3355. Support inline configuration for...

2018-03-20 Thread zjffdu
GitHub user zjffdu opened a pull request: https://github.com/apache/zeppelin/pull/2883 ZEPPELIN-3355. Support inline configuration for user session ### What is this PR for? Generic ConfInterpreter only support process level configuration, this PR is trying to introduce

[GitHub] zeppelin issue #2870: [ZEPPELIN-3335] trim property name of interpreter sett...

2018-03-20 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2870 @rockiee281 seems you didn't do rebase correctly , or you can create another PR ---

[jira] [Created] (ZEPPELIN-3355) Support inline configuration for user session

2018-03-20 Thread Jeff Zhang (JIRA)
Jeff Zhang created ZEPPELIN-3355: Summary: Support inline configuration for user session Key: ZEPPELIN-3355 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3355 Project: Zeppelin Issue

[GitHub] zeppelin issue #2881: ZEPPELIN-3313 z.getInterpreterContext().out().clear() ...

2018-03-20 Thread r-kamath
Github user r-kamath commented on the issue: https://github.com/apache/zeppelin/pull/2881 @zjffdu currently data update and refresh timing in ui grid is not guaranteed. This issue persists only when the interval is below 1sec. I modified your example with 2sec delay and the rendering

[GitHub] zeppelin pull request #2876: ZEPPELIN-3344. Revert comments in queries in JD...

2018-03-20 Thread mebelousov
GitHub user mebelousov reopened a pull request: https://github.com/apache/zeppelin/pull/2876 ZEPPELIN-3344. Revert comments in queries in JDBC interpreter ### What is this PR for? The original purpose of https://github.com/apache/zeppelin/pull/2158 was correct processing of

[GitHub] zeppelin pull request #2876: ZEPPELIN-3344. Revert comments in queries in JD...

2018-03-20 Thread mebelousov
Github user mebelousov closed the pull request at: https://github.com/apache/zeppelin/pull/2876 ---

[GitHub] zeppelin issue #2716: [ZEPPELIN-3123] Fix to create pid per each interpreter

2018-03-20 Thread mebelousov
Github user mebelousov commented on the issue: https://github.com/apache/zeppelin/pull/2716 @woowahan-jaehoon thank you! Perhaps #2780 will delete all unused pid files and you can add the port to the pid filename. ---

[GitHub] zeppelin pull request #2882: ZEPPELIN-3290. Unnecessary message 'IPython is ...

2018-03-20 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2882 ---

[GitHub] zeppelin pull request #2882: ZEPPELIN-3290. Unnecessary message 'IPython is ...

2018-03-20 Thread zjffdu
GitHub user zjffdu opened a pull request: https://github.com/apache/zeppelin/pull/2882 ZEPPELIN-3290. Unnecessary message 'IPython is available, use IPython for PySparkInterpreter' ### What is this PR for? Trivial PR for removing the unnecessary message ###

[jira] [Created] (ZEPPELIN-3354) Change pull mode to push mode for the communication between ZeppelinServer to Interpreter Process

2018-03-20 Thread Jeff Zhang (JIRA)
Jeff Zhang created ZEPPELIN-3354: Summary: Change pull mode to push mode for the communication between ZeppelinServer to Interpreter Process Key: ZEPPELIN-3354 URL: