[GitHub] zeppelin issue #3248: [ZEPPELIN-3878] Browser IE 11 not loading Note

2018-12-05 Thread vinayakshedgeri
Github user vinayakshedgeri commented on the issue:

https://github.com/apache/zeppelin/pull/3248
  
> BTW, I've got the exact same issue like you when I build branch-0.8. 
Please wait a min to investigate this issue.
@jongyoul  any update one this



---


[GitHub] zeppelin issue #3248: [ZEPPELIN-3878] Browser IE 11 not loading Note

2018-12-03 Thread vinayakshedgeri
Github user vinayakshedgeri commented on the issue:

https://github.com/apache/zeppelin/pull/3248
  
> Thank you. BTW, could you please enable your travis-ci to test your PR? 
:-)

Can you help me to run the PR ,its my first contribution and if you know 
any doc/wiki that will fine

I tried to do a build first one failed so triggering again
https://travis-ci.org/vinayakshedgeri/zeppelin/builds/462919073
 


---


[GitHub] zeppelin pull request #3248: [ZEPPELIN-3878] Browser IE 11 not loading Note

2018-12-03 Thread vinayakshedgeri
GitHub user vinayakshedgeri opened a pull request:

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

[ZEPPELIN-3878] Browser IE 11  not loading Note

### What is this PR for?
Previous version of the zeppelin supports IE 11 when  we updated to 
zeppelin 0.8.0, Zeppelin web was not loading properly saying the error promise 
undefined .


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

### Todos
* [ ] - 

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


### How should this be tested?
click on any of the existing note in IE 11 browser. 
Observation  : Note not loading or not reponsive
you can check the error in the console
* First time? Setup Travis CI as described on 
https://zeppelin.apache.org/contribution/contributions.html#continuous-integration
* Strongly recommended: add automated unit tests for any new or changed 
behavior
* Outline any manual steps to test the PR here.

### Screenshots (if appropriate)

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


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

$ git pull https://github.com/vinayakshedgeri/zeppelin branch-0.8

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

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


commit a95be7406a335809491c572b6de29e222c96f19e
Author: Vinay 
Date:   2018-12-03T10:43:43Z

[ZEPPELIN-3878]  IE 11  not loading Note

Unable to load zeppelin note at IE 11 promise undefined




---


[GitHub] zeppelin pull request #3247: [ZEPPELIN-3878] Unable to load zeppelin 0.8.0 n...

2018-12-02 Thread vinayakshedgeri
Github user vinayakshedgeri closed the pull request at:

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


---


[GitHub] zeppelin pull request #3247: [ZEPPELIN-3878] Unable to load zeppelin 0.8.0 n...

2018-12-02 Thread vinayakshedgeri
GitHub user vinayakshedgeri opened a pull request:

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

[ZEPPELIN-3878] Unable to load zeppelin 0.8.0 note at IE 11

### What is this PR for?
Previous version of the zeppelin supports IE 11 when  we updated to 
zeppelin 0.8.0, Zeppelin web was not loading properly saying the error promise 
undefined .


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

### Todos
* [ ] - 

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


### How should this be tested?
click on any of the existing note in IE 11 browser. 
Observation  : Note not loading or not reponsive
you can check the error in the console
* First time? Setup Travis CI as described on 
https://zeppelin.apache.org/contribution/contributions.html#continuous-integration
* Strongly recommended: add automated unit tests for any new or changed 
behavior
* Outline any manual steps to test the PR here.

### Screenshots (if appropriate)

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


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

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

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

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


commit d182a0ed4f44486a92d9c72c3b096470a9c6cbf6
Author: Savalek 
Date:   2018-04-19T12:42:39Z

[ZEPPELIN-3417] fix dependency in zeppelin-web/package.json

### What is this PR for?
The address of the dependency in zeppelin-web/package.json changed from 
"shahata/angular-viewport-watch" to "wix/angular-viewport-watch"

[LINK](https://github.com/wix/angular-viewport-watch) to current repository 
of angular-viewport-watch

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

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

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

Author: Savalek 

Closes #2934 from Savalek/ZEPPELIN-3417 and squashes the following commits:

1cd64aeb1 [Savalek] fix dependency in zeppelin-web/package.json

commit 75dd1958ec195e88c5c0a71dfc56c75fcf6ffaad
Author: Jongyoul Lee 
Date:   2018-04-23T06:43:45Z

ZEPPELIN-3411 Long running logic inside synchronized block in 
InterpreterSettingManager

### What is this PR for?
Removing redundant synchronized code to avoid blocking other logics.

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

### Todos
* [x] - Change synchronized block to read/write lock

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

### How should this be tested?
* Current tests should be passed

### Screenshots (if appropriate)

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

Author: Jongyoul Lee 

Closes #2935 from jongyoul/ZEPPELIN-3411 and squashes the following commits:

3b90155b9 [Jongyoul Lee] Removed unused `import` statements Simplified 
`for` loop
4691301af [Jongyoul Lee] Removed lock/synchronized codes because 
interpreterSettings already is implemented by currentHashMap
24be69298 [Jongyoul Lee] Removed all synchronized blocks and replace them 
to read/write lock

commit 5fdbd5338293dc32f238c834036a7af409d25986
Author: Jongyoul Lee 
Date:   2018-04-23T07:15:21Z

ZEPPELIN-3382 Installing interpreter from UI

### What is this PR for?
Installing a new interpreter from helium's UI. You just can click enable 
button to download a new interpreter you want to install

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

### Todos
* [x] - Changed frontend side to enable users to click an install button
* [x] - Added status message for installation
* [x] - Added Backend logic to handle an interpreter installation

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

### How should this be tested?
1. Make sure you don't have `solr` interpreter in an interpreter create menu
1. Move to Helium/Interpreter menu and click the `solr` enable button
1. See the messages for downloading it and having downloaded it
1. Check interpreter creation menu to make sure `solr` exists

### Screenshots (if appropriate)

![image](https://user-images.githubusercontent.com/3612566/38