[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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`? ---

[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

[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

[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

[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 -

[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

[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

[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

[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

[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