[GitHub] zeppelin issue #2784: Add exclusion to hadoop-aws dependency.

2018-02-09 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/2784
  
Some tests fail, could you retrigger the failed test ?


---


[GitHub] zeppelin issue #2784: Add exclusion to hadoop-aws dependency.

2018-02-09 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/2784
  
LGTM, wait for CI pass


---


[GitHub] zeppelin pull request #2784: Add exclusion to hadoop-aws dependency.

2018-02-09 Thread wilsonr990
GitHub user wilsonr990 opened a pull request:

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

Add exclusion to hadoop-aws dependency.

### What is this PR for?
Fix dependency error caused by different versions of same library

### What type of PR is it?
Bug Fix

### Todos
* [ ZEPPELIN-3217] - Task

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

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

### 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/wilsonr990/zeppelin ZEPPELIN-3217

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

https://github.com/apache/zeppelin/pull/2784.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 #2784


commit 9f3f7e3922104112b7831a977649d99c0913652c
Author: guevara 
Date:   2018-02-09T21:09:31Z

Add exclusion to hadoop-aws dependency.




---


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

2018-02-09 Thread felixcheung
Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/2750
  
Well, my question on the download link
https://github.com/apache/zeppelin/pull/2750#issuecomment-363663111







---


[GitHub] zeppelin pull request #2783: ZEPPELIN-3150. Fixed Checkstyle errors and warn...

2018-02-09 Thread HorizonNet
GitHub user HorizonNet opened a pull request:

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

ZEPPELIN-3150. Fixed Checkstyle errors and warnings in the bigquery module

### What is this PR for?
Fixed the Checkstyle errors and warnings in the bigquery module.

### What type of PR is it?
Improvement

### Todos
* [ ] - Task

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

### 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-3150

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

https://github.com/apache/zeppelin/pull/2783.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 #2783


commit c6be5bed5c2c8d4bd3df88fd008c6515c205d630
Author: Jan Hentschel 
Date:   2018-02-09T17:05:59Z

ZEPPELIN-3150. Fixed Checkstyle errors and warnings in the bigquery module




---


[GitHub] zeppelin issue #2782: [ZEPPELIN-2729] Paragraph numbering

2018-02-09 Thread masyan
Github user masyan commented on the issue:

https://github.com/apache/zeppelin/pull/2782
  
there is Title of paragraph for that


---


[jira] [Created] (ZEPPELIN-3217) Dependency error when loading zeppelin inside of EMR environment

2018-02-09 Thread Wilson Guevara (JIRA)
Wilson Guevara created ZEPPELIN-3217:


 Summary: Dependency error when loading zeppelin inside of EMR 
environment
 Key: ZEPPELIN-3217
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3217
 Project: Zeppelin
  Issue Type: Bug
Affects Versions: 0.8.0
Reporter: Wilson Guevara
 Attachments: pastedImage.png

The  zeppelin functionality of S3NotebookRepo fails with the attached 
stacktrace.

*Environment:*

It was noticed when testing a dockerized zeppelin (*branch-0.8)* inside of an 
EMR.

*Explanation:*

The error points to a lacking method inside of com.amazonaws:aws-java-sdk 
dependency. And, when we listed the zeppelin dependencies we saw two different 
versions of this dependency:

 *- _com.amazonaws:aws-java-sdk:1.7.4_*

 *- _com.amazonaws:aws-java-sdk-core:1.10.62_*

 *- _com.amazonaws:aws-java-sdk-kms:1.10.62_*

 *- _com.amazonaws:aws-java-sdk-s3:1.10.62_*

The first dependency has the oldest version 1.7.4 wish doesn't include the 
stack-trace specified method isInRegionOptimizeModeEnabled(). This dependency 
is added indirectly by another dependency: 
*_org.apache.hadoop:hadoop-aws:2.7.3_*

*Posible solutions:*

We thing this issue was added when the changes in 
[ZEPPELIN-3184|https://issues.apache.org/jira/browse/ZEPPELIN-3184] were added.

In our environment we solved it by excluding the conflicting dependency:

{{  }}
{{    com.amazonaws}}
{{    aws-java-sdk}}
{{  }}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[GitHub] zeppelin issue #2770: ZEPPELIN-3207. fix search content not appearing

2018-02-09 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2770
  
Tested and fix works well. But it's getting build failure with following 
error.

```
[INFO] 
[INFO] /zeppelin/zeppelin-web/src/app/search/result-list.controller.js
[INFO]   151:32  error  Extra semicolon  semi
[INFO] 
```

Could you try remove semicolon?


---


[GitHub] zeppelin issue #2776: Bigquery timestamp formatting fix

2018-02-09 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2776
  
@srikgn Could you try include commits related to this contribution only?


---


[GitHub] zeppelin issue #2775: Googlecloud storage notebook repo

2018-02-09 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2775
  
Thanks for the contribution.
But this pullrequests includes unrelated commits to this contribution. 
@srikgn Could you try include commits related to this contribution only?


---


[GitHub] zeppelin issue #2763: [ZEPPELIN-3194][NEW-INTERPRETER] SAP Universe interpre...

2018-02-09 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2763
  
Cool! Looks great to me.


---


[GitHub] zeppelin pull request #2774: [ZEPPELIN-3212] delete extra ">" in notebook-ac...

2018-02-09 Thread asfgit
Github user asfgit closed the pull request at:

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


---


[GitHub] zeppelin issue #2774: [ZEPPELIN-3212] delete extra ">" in notebook-actionBar...

2018-02-09 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/2774
  
Merge to master and branch-0.8


---


[GitHub] zeppelin pull request #2782: Paragraph numbering

2018-02-09 Thread timovwb
GitHub user timovwb opened a pull request:

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

Paragraph numbering

### What is this PR for?
Added a button on the top toolbar of the notebook to toggle paragraph 
numbering. It adds a "Paragraph N" on the top-right of each paragraph, mainly 
useful when collaborating and guiding others to the right place.


### What type of PR is it?
Feature

### What is the Jira issue?
[ZEPPELIN-2729](https://issues.apache.org/jira/browse/ZEPPELIN-2729)

### Screenshots
[gfycat mp4 video](https://gfycat.com/LeafyAntiqueCrab)

### 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/timovwb/zeppelin paragraph-numbering

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

https://github.com/apache/zeppelin/pull/2782.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 #2782


commit c1974362597c59ab081f76be1de8c21f3050b0ba
Author: Timo Vanwynsberghe 
Date:   2018-02-02T14:19:34Z

Added option to toggle paragraph numbering

commit 09c8141437195842733af953260d7b48d8810151
Author: Timo Vanwynsberghe 
Date:   2018-02-02T15:02:36Z

Removed a newline by mistake




---


[GitHub] zeppelin issue #2768: ZEPPELIN-3142. Fixed Checkstyle errors and warnings in...

2018-02-09 Thread HorizonNet
Github user HorizonNet commented on the issue:

https://github.com/apache/zeppelin/pull/2768
  
@zjffdu Here's the 
[link](https://travis-ci.org/ultratendency/zeppelin/builds/339087714). I had to 
rebase this PR due to recent changes.


---


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

2018-02-09 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/2750
  
ping @felixcheung 


---


[GitHub] zeppelin issue #2768: ZEPPELIN-3142. Fixed Checkstyle errors and warnings in...

2018-02-09 Thread zjffdu
Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/2768
  
@HorizonNet I don't see the green build, could you point me the link ?


---