[GitHub] zeppelin issue #3181: ZEPPELIN-3633. ZeppelinContext Not Found in yarn-clust...

2018-09-18 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/3181
  
@zjffdu Sure. I'm willing to do it.


---


[GitHub] zeppelin issue #3180: [ZEPPELIN-3781] Bump up version of npm packages

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

https://github.com/apache/zeppelin/pull/3180
  
Is it necessary to backport to branch-0.8 ? Does it fix some critical issue 
?


---


[GitHub] zeppelin issue #3180: [ZEPPELIN-3781] Bump up version of npm packages

2018-09-18 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/3180
  
Sure, thanks for the review will merge this soon to both master and 
branch-0.8


---


[GitHub] zeppelin issue #3181: ZEPPELIN-3633. ZeppelinContext Not Found in yarn-clust...

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

https://github.com/apache/zeppelin/pull/3181
  
@jongyoul Could you help review and verify it ? Thanks 


---


[GitHub] zeppelin pull request #3181: ZEPPELIN-3633. ZeppelinContext Not Found in yar...

2018-09-18 Thread zjffdu
GitHub user zjffdu opened a pull request:

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

ZEPPELIN-3633. ZeppelinContext Not Found in yarn-cluster Mode

### What is this PR for?
This issue is due to classpath in cluster mode. Because in cluster mode, 
driver run in the node of yarn cluster which don't have zeppelin installed. 
This PR fix this issue by updating the classpath of spark repl. 

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

### Todos
* [ ] - Task

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

### How should this be tested?
* Manually tested. Unfortunately this only happens on multiple node 
cluster, no unit test can be added for this scenario. 

### Screenshots (if appropriate)

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


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

$ git pull https://github.com/zjffdu/zeppelin ZEPPELIN-3633

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

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


commit 73ed8702a66456ed4066f339c84957ea7239b115
Author: Jeff Zhang 
Date:   2018-09-19T03:21:50Z

ZEPPELIN-3633. ZeppelinContext Not Found in yarn-cluster Mode




---


[GitHub] zeppelin issue #3180: [ZEPPELIN-3781] Bump up version of npm packages

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

https://github.com/apache/zeppelin/pull/3180
  
Thanks @prabhjyotsingh LGTM


---


[GitHub] zeppelin issue #3180: [ZEPPELIN-3781] Bump up version of npm packages

2018-09-18 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/3180
  
Changes consist of two files; 
 - The larger one being package-lock.json; It stores an exact, versioned 
dependency tree rather than using starred versioning like package.json itself 
(e.g. 1.0.*). This means you can guarantee the dependencies for other 
developers or prod releases, etc. It also has a mechanism to lock the tree but 
generally will regenerate if package.json changes. 
https://docs.npmjs.com/files/package-lock.json
 - package.json contains the actual changes that needs upgrade.


---


[GitHub] zeppelin issue #3180: [ZEPPELIN-3781] Bump up version of npm packages

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

https://github.com/apache/zeppelin/pull/3180
  
oops, it seems a large change, not sure why node npm package upgrade needs 
so large change. I am not an frontend expert, just need some explanation. 😄 


---


[jira] [Created] (ZEPPELIN-3783) java.lang.RuntimeException: Can not convert this type of property: class com.google.gson.internal.LinkedTreeMap

2018-09-18 Thread Saif Addin (JIRA)
Saif Addin created ZEPPELIN-3783:


 Summary: java.lang.RuntimeException: Can not convert this type of 
property: class com.google.gson.internal.LinkedTreeMap
 Key: ZEPPELIN-3783
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3783
 Project: Zeppelin
  Issue Type: Bug
  Components: zeppelin-interpreter
Affects Versions: 0.8.0
Reporter: Saif Addin


Hi, using Hortonworks here. After restarting zeppelin from Ambari, I get
{panel:title=Error 503}


 
h2. HTTP ERROR: 503

Problem accessing /. Reason:

Service Unavailable

 
{panel}
 

Error in logs is as follows
{code:java}
 INFO [2018-09-18 19:00:58,938] ({main} 
FileSystemConfigStorage.java[loadInterpreterSettings]:75) - Load Interpreter 
Setting from file: 
hdfs://ip-172-31-11-62.us-west-1.compute.internal:8020/user/zeppelin/conf/interpreter.json
 
WARN [2018-09-18 19:00:59,081] ({main} ContextHandler.java[log]:2177) - 
unavailable

MultiException stack 1 of 1
java.lang.RuntimeException: Can not convert this type of property: class 
com.google.gson.internal.LinkedTreeMap
    at 
org.apache.zeppelin.interpreter.InterpreterSetting.convertInterpreterProperties(InterpreterSetting.java:925)

at 
org.apache.zeppelin.interpreter.InterpreterSettingManager.loadFromFile(InterpreterSettingManager.java:221)
    at 
org.apache.zeppelin.interpreter.InterpreterSettingManager.init(InterpreterSettingManager.java:337)
    at 
org.apache.zeppelin.interpreter.InterpreterSettingManager.(InterpreterSettingManager.java:171)
    at 
org.apache.zeppelin.interpreter.InterpreterSettingManager.(InterpreterSettingManager.java:131)
    at 
org.apache.zeppelin.server.ZeppelinServer.(ZeppelinServer.java:151)
    at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
    at 
sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
    at 
sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
    at java.lang.reflect.Constructor.newInstance(Constructor.java:423)
    at 
org.glassfish.hk2.utilities.reflection.ReflectionHelper.makeMe(ReflectionHelper.java:1375)
    at 
org.jvnet.hk2.internal.Utilities.justCreate(Utilities.java:1007){code}



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


[GitHub] zeppelin issue #3180: [ZEPPELIN-3781] Bump up version of npm packages

2018-09-18 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/3180
  
CI green.
@zjffdu @felixcheung can you help review this.


---


[jira] [Created] (ZEPPELIN-3782) Progress animation uses an inordinate amount of system resources.

2018-09-18 Thread johan be (JIRA)
johan be created ZEPPELIN-3782:
--

 Summary: Progress animation uses an inordinate amount of system 
resources.
 Key: ZEPPELIN-3782
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3782
 Project: Zeppelin
  Issue Type: Bug
  Components: GUI
Affects Versions: 0.7.3
Reporter: johan be


The blue progress-bar animation, which is displayed whenever a paragraph is 
evaluated, uses an inordinate amount of resources on my system.

Running this animation consumes the entire resources of 1 CPU-core on Firefox 
and 2 cores in Chromium.



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


[GitHub] zeppelin pull request #3180: [ZEPPELIN-3781] Bump up version of npm packages

2018-09-18 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request:

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

[ZEPPELIN-3781] Bump up version of npm packages

### What is this PR for?
The goal of this PR is to track bump up version on babel, karma-coverage, 
protractor, and webpack-dev-server packages.


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

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

### How should this be tested?
CI shoud be green 

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


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

$ git pull https://github.com/prabhjyotsingh/zeppelin ZEPPELIN-3781

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

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


commit 4e0c5fe8f4872b2308da570395a23eee04021c3c
Author: Prabhjyot Singh 
Date:   2018-09-18T11:12:52Z

ZEPPELIN-3781: Bump up version of npm packages

Change-Id: I5db4440cd0f980d97919aeef913a32849782559f




---


[jira] [Created] (ZEPPELIN-3781) Bump up version of npm packages

2018-09-18 Thread Prabhjyot Singh (JIRA)
Prabhjyot Singh created ZEPPELIN-3781:
-

 Summary: Bump up version of npm packages
 Key: ZEPPELIN-3781
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3781
 Project: Zeppelin
  Issue Type: Bug
Reporter: Prabhjyot Singh


The goal of this JIRA is to track bump up version on babel, karma-coverage, 
protractor, and webpack-dev-server packages.



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


[GitHub] zeppelin issue #3163: ZEPPELIN-2619. Save note in ${notename_noteid}.zpln in...

2018-09-18 Thread mebelousov
Github user mebelousov commented on the issue:

https://github.com/apache/zeppelin/pull/3163
  
@zjffdu 
No, I didn't see the popup with error explanation for user.

I have tested a lot with Russian symbols. Seems that Zeppelin works fine.

Thank you for this improvement. PR could be merged.


---


[GitHub] zeppelin issue #3179: [ZEPPELIN-3773] - add check permission on write.

2018-09-18 Thread mebelousov
Github user mebelousov commented on the issue:

https://github.com/apache/zeppelin/pull/3179
  
@Savalek 
You're right. I fix my shiro.ini.

But I still see a huge amount of background queries (up to 100 per second) 
connected with Helium.
I can reproduce the issue in development mode.
Tested on Ubuntu 16.04, Chromium (Version 69.0.3497.92) and Firefox Quantum 
62.0
Check please.


---


[GitHub] zeppelin issue #2373: [ZEPPELIN-2598] Securing Zeppelin with OpenID Connect

2018-09-18 Thread ajaygk95
Github user ajaygk95 commented on the issue:

https://github.com/apache/zeppelin/pull/2373
  
> 
> 
> @SarunasG @1ambda @khalidhuseynov @andreaTP Hello, is there any doc of 
details for me to add libs in 0.7.3 and by changing the shiro.ini config file 
to support open id?

Hello @xixikaikai ,
I am using zeppelin 0.8.0 and with the below changes i am able to integrate 
it with keycloak.

Dependencies to be added in zeppelin-0.8.0 lib folder:
buji-pac4j-3.0.0.jar
lang-tag-1.4.3.jar
json-smart-1.3.1.jar
commons-lang3-3.5.jar
commons-collections4-4.1.jar
pac4j-core-2.3.1.jar
oauth2-oidc-sdk-5.24.2.jar
mail-1.4.7.jar
shiro-crypto-hash-1.4.0.jar
shiro-crypto-core-1.4.0.jar
shiro-crypto-cipher-1.4.0.jar
shiro-core-1.4.0.jar
shiro-config-ogdl-1.4.0.jar
shiro-config-core-1.4.0.jar
shiro-cache-1.4.0.jar
pac4j-oidc-2.3.1.jar
slf4j-api-1.7.25.jar
shiro-web-1.4.0.jar
shiro-lang-1.4.0.jar
shiro-event-1.4.0.jar


I have added the shiro.ini conf and dependency in the attachment.
[dep.txt](https://github.com/apache/zeppelin/files/2391716/dep.txt)
 



---


[GitHub] zeppelin issue #3157: [ZEPPELIN-3654] - New Hazelcast Jet interpreter

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

https://github.com/apache/zeppelin/pull/3157
  
Thanks @selvinsource Could you fix the conflict ?


---


[GitHub] zeppelin pull request #3176: ZEPPELIN-3558. Refactor Scheduler component

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

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


---


[GitHub] zeppelin issue #3163: ZEPPELIN-2619. Save note in ${notename_noteid}.zpln in...

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

https://github.com/apache/zeppelin/pull/3163
  
@mebelousov Do you see popup windows saying `Fail to create note, cause: 
Note name can not contain '..'` ? This is by design. 

Regarding 2, 3, this is a known issue, this is due to whether we should 
allow duplicate note name under folder. I plan to discuss it in community, will 
fix it in follow up PR.

Regarding 6,7, I plan to fix it in follow up PR as they are not trivial to 
fix. 




---