[GitHub] zeppelin pull request #1552: [HOTFIX] bump flink to 1.1.3
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 GrantDate: 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...
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...
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...
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 ...
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...
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...
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)
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...
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...
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
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.
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...
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...
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...
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 HentschelDate: 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...
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 JiangDate: 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
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
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 ZhangDate: 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
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
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)
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
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
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. ---