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

2018-03-08 Thread SleepyThread
Github user SleepyThread commented on the issue: https://github.com/apache/zeppelin/pull/2817 I have created a [clean pull request](https://github.com/apache/zeppelin/pull/2852). Closing this pull request. ---

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

2018-03-06 Thread SleepyThread
Github user SleepyThread commented on the issue: https://github.com/apache/zeppelin/pull/2817 @Tagar - It is using the revision feature from org.apache.zeppelin.notebook.repo.GitNotebookRepo in nutshell, a snapshot is like a CNAME to the revision. People can share the CNAME to

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

2018-03-02 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/2817 Interesting feature, thanks @SleepyThread for the contribution. ---

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

2018-03-01 Thread Tagar
Github user Tagar commented on the issue: https://github.com/apache/zeppelin/pull/2817 @SleepyThread - great feature! Building on @felixcheung's comment, it would be good if Snapshot feature would use org.apache.zeppelin.notebook.repo.GitNotebookRepo to store a

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

2018-02-27 Thread SleepyThread
Github user SleepyThread commented on the issue: https://github.com/apache/zeppelin/pull/2817 @felixcheung this feature just adds a mapping of Snapshot ID to revision id in storage like git. This helps in sharing a static URL by user /notebook/**{Notebook ID}**/view/**{Snapshot

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

2018-02-26 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2817 hmm, interesting. how is this different or work with notebook revision on storage like git? ---

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

2018-02-26 Thread mebelousov
Github user mebelousov commented on the issue: https://github.com/apache/zeppelin/pull/2817 @SleepyThread thank you! This is a nice feature. If the PR will be approved it would be great to have next functionality: - limit the number of snapshots and/or life time of