[GitHub] zeppelin pull request #1552: [HOTFIX] bump flink to 1.1.3

2016-10-22 Thread rawkintrevo
GitHub user rawkintrevo opened a pull request:

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

[HOTFIX] bump flink to 1.1.3

### What is this PR for?
A few sentences describing the overall goals of the pull request's commits.
First time? Check out the contributing guide - 
https://zeppelin.apache.org/contribution/contributions.html


### What type of PR is it?
[Bug Fix | Improvement | Feature | Documentation | Hot Fix | Refactoring]

### Todos
* [ ] - Task

### What is the Jira issue?
* Open an issue on Jira https://issues.apache.org/jira/browse/ZEPPELIN/
* Put link here, and add [ZEPPELIN-*Jira number*] in PR title, eg. 
[ZEPPELIN-533]

### How should this be tested?
Outline the steps to test the PR here.

### 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/rawkintrevo/incubator-zeppelin patch-1

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

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


commit 248a57b194569c1a0b9b427a26dfa55154b5618f
Author: Trevor Grant 
Date:   2016-10-23T04:40:22Z

[HOTFIX] bump flink to 1.1.3




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1518: [ZEPPELIN-1549] Change NotebookID variable name to Not...

2016-10-22 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1518
  
@hyonzin Could you rebase please? Let me know if you need any help.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1543: Add missing TOC(Table of Contents) in install_w...

2016-10-22 Thread asfgit
Github user asfgit closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1542: [MINOR][DOC] Fix typo (re-open PR #1541 with re...

2016-10-22 Thread asfgit
Github user asfgit closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1530: [Zeppelin-1555] Eliminate prefix in PythonInterpreter ...

2016-10-22 Thread vectorijk
Github user vectorijk commented on the issue:

https://github.com/apache/zeppelin/pull/1530
  
@Leemoonsoo @bzz Thanks for the review!


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1530: [Zeppelin-1555] Eliminate prefix in PythonInter...

2016-10-22 Thread asfgit
Github user asfgit closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1543: Add missing TOC(Table of Contents) in install_with_fli...

2016-10-22 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1543
  
LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1542: [MINOR][DOC] Fix typo (re-open PR #1541 with rebase)

2016-10-22 Thread anthonycorbacho
Github user anthonycorbacho commented on the issue:

https://github.com/apache/zeppelin/pull/1542
  
Changes LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1550: [ZEPPELIN-912] Apply new mechanism to GeodeOqlInterpre...

2016-10-22 Thread vectorijk
Github user vectorijk commented on the issue:

https://github.com/apache/zeppelin/pull/1550
  
@jongyoul Thanks for the review!


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1467: [ZEPPELIN-1488] JDBC Interpreter throws error w...

2016-10-22 Thread asfgit
Github user asfgit closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1551: [ZEPPELIN-429] Replaced explicit types with diamonds

2016-10-22 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1551
  
Thank for the contribution. I like this kind of contribution to make code 
clean. LGTM. I'll merge it.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1302: [ZEPPELIN-1306] Interpreter restarts on a note.

2016-10-22 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1302
  
Can you add consideration for the multi-tenancy? If Shiro is enabled, 
current implementation will disable all of users' interpreters


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1550: [ZEPPELIN-912] Apply new mechanism to GeodeOqlInterpre...

2016-10-22 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1550
  
Thanks for the contribution. LGTM. 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1546: [ Hotfix ] Incorrect (per user) setup ui for "c...

2016-10-22 Thread asfgit
Github user asfgit closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1551: [ZEPPELIN-429] Replaced explicit types with dia...

2016-10-22 Thread HorizonNet
GitHub user HorizonNet opened a pull request:

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

[ZEPPELIN-429] Replaced explicit types with diamonds

### What is this PR for?
Changed all occurrences of explicit types to diamonds in the different 
modules.

### What type of PR is it?
Improvement

### Todos
-

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

### How should this be tested?
Run the normal Maven build process.

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

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

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


commit b56bf24358b30ce6e387e5678b5977e516784e53
Author: Jan Hentschel 
Date:   2016-10-22T14:40:35Z

[ZEPPELIN-429] Replaced explicit types with diamonds

- Changed all occurences of explicity types to diamonds in the different
  modules




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1550: [ZEPPELIN-912] Apply new mechanism to GeodeOqlI...

2016-10-22 Thread vectorijk
GitHub user vectorijk opened a pull request:

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

[ZEPPELIN-912] Apply new mechanism to GeodeOqlInterpreter

### What is this PR for?
This PR applies the new interpreter registration mechanism to Geode Oql 
interpreter.

### What type of PR is it?
Improvement

### Todos
Added `interpreter-setting.json`

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

### How should this be tested?
Test locally.

### 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/vectorijk/zeppelin zeppelin-912

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

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


commit 5dfad61a788af3d386c83f1516768edf0534dd93
Author: Kai Jiang 
Date:   2016-10-22T13:13:11Z

Apply new mechanism to GeodeOqlInterpreter




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1549: ZEPPELIN-1547. Paragraph text should be trimmed

2016-10-22 Thread zjffdu
Github user zjffdu closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin pull request #1549: ZEPPELIN-1547. Paragraph text should be trimmed

2016-10-22 Thread zjffdu
GitHub user zjffdu reopened a pull request:

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

ZEPPELIN-1547. Paragraph text should be trimmed

### What is this PR for?
The paragraph text should be trimmed, otherwise repl name can not be 
correctly recognized. 


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

### Todos
* [ ] - Task

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

### How should this be tested?
Unit test is added

### 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/zjffdu/zeppelin ZEPPELIN-1574

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

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


commit 40194e7bcaa6d69ee1ab94e28506319edba84935
Author: Jeff Zhang 
Date:   2016-10-22T05:31:08Z

ZEPPELIN-1547. Paragraph text should be trimmed

commit 5378467286062d754275116fb3aa091b85ab5a27
Author: Jeff Zhang 
Date:   2016-10-22T06:01:23Z

fix unit test




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1517: [MINOR] ui improvement

2016-10-22 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1517
  
@baekhoseok Thank you for the contribution! Do you mind to update 
`docs/assets/themes/zeppelin/img/ui-img/interpreter_menu.png` again?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1517: [MINOR] ui improvement

2016-10-22 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1517
  
@AhyoungRyu the issue you raised is not caused by this pr but #1501. Can 
you try after clean bower cache(`bower cache clean`) and remove 
`bower_components` directory under `zeppelin-web`?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1542: [MINOR][DOC] Fix typo (re-open PR #1541 with rebase)

2016-10-22 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1542
  
@laesunk I made a PR to your repository to fix more typos, let me merge 
this PR after you merge mine to yours.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1544: spelling error

2016-10-22 Thread minahlee
Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1544
  
Thank you for the contribution. Having description would help reviewers to 
understand what you are trying to do. I believe you will do it from next time :)
Plus do you mind close #1547? I think it duplicates this one.
Meanwhile I will merge this in next 24 hours


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] zeppelin issue #1497: [ZEPPELIN-1233] Support Redis interpreter

2016-10-22 Thread anthonycorbacho
Github user anthonycorbacho commented on the issue:

https://github.com/apache/zeppelin/pull/1497
  
@HeartSaVioR yeah i would love to have a class that manipulate redis cmd 
and handle null, ussually i am not a huge fan of returning `null`, i would 
prefer to return an `EMPTY` type instead.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---