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

[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 #2764: ZEPPELIN-3146. Fixed Checkstyle errors in alluxio modu...

[zjffdu]

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/2764
  
@felixcheung Right, I currently turn off checkstyle, after all the modules 
fix the code style, will turn it on in root pom file


---

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

[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 pull request #2738: [ZEPPELIN-3182] Support saving notebooks to Goo...

[karth295]

Github user karth295 closed the pull request at:

https://github.com/apache/zeppelin/pull/2738


---

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

[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 pull request #2738: [ZEPPELIN-3182] Support saving notebooks to Goo...

[karth295]

GitHub user karth295 reopened a pull request:

https://github.com/apache/zeppelin/pull/2738

[ZEPPELIN-3182] Support saving notebooks to Google Cloud Storage

### What is this PR for?

Support saving notebooks to Google Cloud Storage, similar to 
implementations for S3 and Azure. It uses the same authentication mechanisms as 
the BigQuery interpreter.

I am new to Maven, so please check my work on the pom.xml files. In 
particular, I upgraded Guava to 23.0, which was required for 
`google-cloud-java`. Going through hello-world with my changes seems to work.

Also, I modified the BigQuery interpreter docs to point to the **latest** 
GCS storage docs. Is it more appropriate to pin to the version you are viewing? 
How can I do that?

### What type of PR is it?
Improvement

### Todos
* [Low priority] Support encryption keys

I don't this is particularly important, at least for v1.

### How should this be tested?
* I added unit tests for the core functionality
* I manually tested the authentication instructions (but that could use a 
second pair of eyes)

### Questions:
* Does the licenses files need update?
  * No idea. `google-cloud-java` is Apache 2: 
https://github.com/GoogleCloudPlatform/google-cloud-java/blob/master/LICENSE
* Is there breaking changes for older versions?
  * Nope.
* Does this needs documentation?
  * Yes, and I tried to update the docs (but there are likely other things 
that need to be updated)

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/karth295/zeppelin master

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2738.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2738


commit 8dc819ee4a322d3616f82f6c720eac221fc32ea0
Author: Karthik Palaniappan 
Date:   2018-01-29T03:10:40Z

Unify logic to clear notebook runtime state on load from storage

commit 68b1693568e49e15d47a1d597a97cb5ee70805be
Author: Karthik Palaniappan 
Date:   2018-01-29T03:21:34Z

[ZEPPELIN-3182] Support saving notebooks to Google Cloud Storage




---

[GitHub] zeppelin pull request #2764: ZEPPELIN-3146. Fixed Checkstyle errors in allux...

[HorizonNet]

GitHub user HorizonNet opened a pull request:

https://github.com/apache/zeppelin/pull/2764

ZEPPELIN-3146. Fixed Checkstyle errors in alluxio module

### What is this PR for?
Fix the Checkstyle errors and warning in the **alluxio** module.

### What type of PR is it?
Improvement

### Todos
* [ ] - Task

### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-3146

### How should this be tested?
* CI pass

### Screenshots (if appropriate)

### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/ultratendency/zeppelin ZEPPELIN-3146

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2764.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2764


commit 8269db69bc0e5c127dc598f16e4130d6e0a301cd
Author: Jan Hentschel 
Date:   2018-02-03T18:46:39Z

ZEPPELIN-3146. Fixed Checkstyle errors in alluxio module




---

[GitHub] zeppelin pull request #2763: [ZEPPELIN-3194][NEW-INTERPRETER] SAP Universe i...

[masyan]

GitHub user masyan opened a pull request:

https://github.com/apache/zeppelin/pull/2763

[ZEPPELIN-3194][NEW-INTERPRETER] SAP Universe interpreter

### What is this PR for?
New interpreter, SAP BusinessObjects (Universes). See 
[documentation](https://github.com/masyan/zeppelin/blob/9178d1cc80253b9ae58b988443b619127f42d945/docs/interpreter/sap.md)

### What type of PR is it?
[Feature]

### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-3194

### How should this be tested?
see screenshots

### Screenshots (if appropriate)
Autocomplete + get results

![comletion_and_results](https://user-images.githubusercontent.com/705100/35770106-1fb3e9e6-0937-11e8-9ee4-df0c06a7b6c3.gif)

Errors

![errors](https://user-images.githubusercontent.com/705100/35770108-23de9674-0937-11e8-89e1-a0d3380e88bb.gif)


### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/masyan/zeppelin ZEPPELIN-3194

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/zeppelin/pull/2763.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2763


commit f6b71f9c2a776c32972e14029f79638d364ccb9f
Author: Maksim Reshetov 
Date:   2018-01-31T09:49:44Z

[ZEPPELIN-3194] is null fix

commit c1c09af5c014ca3c6410ed5dbeecc81738188b39
Author: Maksim Reshetov 
Date:   2018-01-31T11:01:59Z

[ZEPPELIN-3194] docs fix + unit tests for util

commit 1aeb7ad0f14c470253e668d32c9bac7757768e2d
Author: Maksim Reshetov 
Date:   2018-01-31T11:11:39Z

[ZEPPELIN-3194] pom license

commit e00ace2e3efdc869c34d9e3bafe353f0eaa45f77
Author: Maksim Reshetov 
Date:   2018-01-31T15:32:30Z

[ZEPPELIN-3194] completer tests

commit 0784ba0303aaf50d438fe18729f219fa5df0e9b1
Author: Maksim Reshetov 
Date:   2018-02-01T14:30:14Z

[ZEPPELIN-3194] docs

commit ce97551b9611574ad3f68f49837a0acdcd218f1b
Author: Maksim Reshetov 
Date:   2018-02-03T14:29:37Z

Merge remote-tracking branch 'upstream/master' into ZEPPELIN-3194

commit 9178d1cc80253b9ae58b988443b619127f42d945
Author: Maksim Reshetov 
Date:   2018-02-03T14:39:16Z

[ZEPPELIN-3194] docs




---

[GitHub] zeppelin issue #2759: ZEPPELIN-3202. Added missing test dependencies in the ...

[HorizonNet]

Github user HorizonNet commented on the issue:

https://github.com/apache/zeppelin/pull/2759
  
I kicked off another Travis build and it seems that these failing tests are 
flakey.


---

[GitHub] zeppelin issue #2750: ZEPPELIN-3108. Support Spark 2.3

[zjffdu]

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/2750
  
@felixcheung , download url is fixed, please help review again. 


---

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

[zjffdu]

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/2760
  
I haven't updated the description, will do that after the PR is ready, It 
is still WIP 


---

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

[masyan]

Github user masyan commented on the issue:

https://github.com/apache/zeppelin/pull/2760
  
@zjffdu 
a strange description of PR )


---

[GitHub] zeppelin pull request #2761: [HOTFIX] Fix spark version mismatch

[zjffdu]

Github user zjffdu closed the pull request at:

https://github.com/apache/zeppelin/pull/2761


---

[GitHub] zeppelin issue #2750: ZEPPELIN-3108. Support Spark 2.3

[felixcheung]

Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/2750
  
The s3 is no longer being used. We need to move to apache mirrors as per 
ASF policy..





---