[GitHub] zeppelin issue #2738: [ZEPPELIN-3182] Support saving notebooks to Google Clo...

2018-02-04 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2738 Thanks @karth295 will merge if no more comments ---

[GitHub] zeppelin issue #2738: [ZEPPELIN-3182] Support saving notebooks to Google Clo...

2018-02-04 Thread karth295
Github user karth295 commented on the issue: https://github.com/apache/zeppelin/pull/2738 Nice, it's green now (though Jenkins didn't update). ---

[GitHub] zeppelin issue #2738: [ZEPPELIN-3182] Support saving notebooks to Google Clo...

2018-02-03 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2738 The failure should due to jar file corrupted, could you clean travis cache and trigger the failed build again ? (I mean only rerun the failed build, not all the travis build matrix) ---

[GitHub] zeppelin issue #2738: [ZEPPELIN-3182] Support saving notebooks to Google Clo...

2018-02-03 Thread karth295
Github user karth295 commented on the issue: https://github.com/apache/zeppelin/pull/2738 @zjffdu do you know what's causing these failure? (I can't tell whether my PR is breaking the tests) ---

[GitHub] zeppelin issue #2738: [ZEPPELIN-3182] Support saving notebooks to Google Clo...

2018-02-03 Thread karth295
Github user karth295 commented on the issue: https://github.com/apache/zeppelin/pull/2738 The new CI failure doesn't seem real to me -- going to close/reopen to make it run again. ---

[GitHub] zeppelin issue #2738: [ZEPPELIN-3182] Support saving notebooks to Google Clo...

2018-01-29 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2738 @karth295 The only concern is guava, if that doens't affect other function, it is fine to me to merge into 0.8 as well ---

[GitHub] zeppelin issue #2738: [ZEPPELIN-3182] Support saving notebooks to Google Clo...

2018-01-29 Thread karth295
Github user karth295 commented on the issue: https://github.com/apache/zeppelin/pull/2738 @zjffdu I noticed that you just bumped the version to 0.9 -- can this feature be integrated into branch-0.8 release as well? ---

[GitHub] zeppelin issue #2738: [ZEPPELIN-3182] Support saving notebooks to Google Clo...

2018-01-28 Thread karth295
Github user karth295 commented on the issue: https://github.com/apache/zeppelin/pull/2738 FYI I finally resolved the dependency issues by using guava 20.0, which is the only version that works for both the `google-cloud-java` libraries as well as Hadoop (the deprecated `Objects`

[GitHub] zeppelin issue #2738: [ZEPPELIN-3182] Support saving notebooks to Google Clo...

2018-01-24 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2738 CI still fails, could you retrigger it again ? Just close and reopen this PR. ---

[GitHub] zeppelin issue #2738: [ZEPPELIN-3182] Support saving notebooks to Google Clo...

2018-01-23 Thread karth295
Github user karth295 commented on the issue: https://github.com/apache/zeppelin/pull/2738 These two commits are rebased over master. ---

[GitHub] zeppelin issue #2738: [ZEPPELIN-3182] Support saving notebooks to Google Clo...

2018-01-23 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2738 @karth295 Please try to rebase your PR, I think the build issue you mentioned above has been fixed. ---

[GitHub] zeppelin issue #2738: [ZEPPELIN-3182] Support saving notebooks to Google Clo...

2018-01-23 Thread karth295
Github user karth295 commented on the issue: https://github.com/apache/zeppelin/pull/2738 The TravisCI failure doesn't look right to me: ``` [WARNING] Dependency convergence error for com.google.guava:guava:15.0 paths to dependency are: