[GitHub] zeppelin pull request #1954: [ZEPPELIN-525] Test failing in zeppelin-interpr...

2017-01-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1954 --- 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 e

[GitHub] zeppelin issue #1958: [MINOR] Prevent log NPE in ResourcePoolUtils.getAllRes...

2017-01-29 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1958 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 fe

[GitHub] zeppelin issue #1954: [ZEPPELIN-525] Test failing in zeppelin-interpreter

2017-01-29 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1954 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 fe

[GitHub] zeppelin issue #1959: [ZEPPELIN-2026] Flaky Test: WebDriverManager.getWebDri...

2017-01-29 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1959 Good plan :-) --- 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

[GitHub] zeppelin issue #1957: Branch 0.6 Groovy Interpreter

2017-01-29 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1957 Thanks for contributing new Groovy interpreter but we didn't manage the branch-0.6 anymore. Can you rebase your contribution onto current master? --- If your project is set up for it, you can re

[GitHub] zeppelin issue #1959: [ZEPPELIN-2026] Flaky Test: WebDriverManager.getWebDri...

2017-01-29 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1959 Merge to master if no more comments --- 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 en

[GitHub] zeppelin pull request #1960: [ZEPPELIN-2030] Select statement doesn't work i...

2017-01-29 Thread del680202
GitHub user del680202 opened a pull request: https://github.com/apache/zeppelin/pull/1960 [ZEPPELIN-2030] Select statement doesn't work in LivySparkSQLInterpreter ### What is this PR for? I try to execute sql query by ``%livy.sql``, I got a error as below ``` ERROR [2017-

[GitHub] zeppelin issue #1940: [ZEPPELIN-2008] Introduce Spell

2017-01-29 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1940 finally, CI is green. :) --- 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 wishe

[GitHub] zeppelin issue #1940: [ZEPPELIN-2008] Introduce Spell

2017-01-29 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1940 Thanks addressing the comments. Tested and it works well as expected. Agree, 1) can be handled in separate issue while this PR already quite big. CI fails on RAT check profile

[GitHub] zeppelin pull request #1955: [ZEPPELIN-2025] Cleanup R Tutorial

2017-01-29 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/1955 --- 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 e

[GitHub] zeppelin pull request #1955: [ZEPPELIN-2025] Cleanup R Tutorial

2017-01-29 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/1955 [ZEPPELIN-2025] Cleanup R Tutorial ### What is this PR for? 1. Remove useless paragraphs in R tutorial 2. Prettify layout I attached screenshots for comparison. See the scre

[GitHub] zeppelin issue #1940: [ZEPPELIN-2008] Introduce Spell

2017-01-29 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/1940 rebased to resolve conflict with master. @Leemoonsoo Thanks for detailed review! I updated code according to your comment. (1) **Couldn't make Spell as display system work:** Let's

[GitHub] zeppelin pull request #1902: [ZEPPELIN-1821] Add HTTP client to elasticsearc...

2017-01-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1902 --- 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 e

[GitHub] zeppelin pull request #1959: [ZEPPELIN-2026] Flaky Test: WebDriverManager.ge...

2017-01-29 Thread Leemoonsoo
GitHub user Leemoonsoo opened a pull request: https://github.com/apache/zeppelin/pull/1959 [ZEPPELIN-2026] Flaky Test: WebDriverManager.getWebDriver() fails with Unable to locate element 'WebSocket Connected' ### What is this PR for? Selenium test sometimes fails with

[GitHub] zeppelin pull request #1955: [ZEPPELIN-2025] Cleanup R Tutorial

2017-01-29 Thread 1ambda
Github user 1ambda closed the pull request at: https://github.com/apache/zeppelin/pull/1955 --- 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 e

[GitHub] zeppelin pull request #1955: [ZEPPELIN-2025] Cleanup R Tutorial

2017-01-29 Thread 1ambda
GitHub user 1ambda reopened a pull request: https://github.com/apache/zeppelin/pull/1955 [ZEPPELIN-2025] Cleanup R Tutorial ### What is this PR for? 1. Remove useless paragraphs in R tutorial 2. Prettify layout I attached screenshots for comparison. See the scre

[GitHub] zeppelin pull request #1958: [MINOR] Prevent log NPE in ResourcePoolUtils.ge...

2017-01-29 Thread Leemoonsoo
GitHub user Leemoonsoo opened a pull request: https://github.com/apache/zeppelin/pull/1958 [MINOR] Prevent log NPE in ResourcePoolUtils.getAllResourcesExcept() ### What is this PR for? ResourcePoolUtils.getAllResourcesExcept() sometimes throws NPE. Which doesn't really need

[GitHub] zeppelin issue #1663: ZEPPELIN-1692: Ability to access Spark jobs UI from th...

2017-01-29 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1663 last CI run fails with ``` Tests run: 9, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 228.978 sec <<< FAILURE! - in org.apache.zeppelin.integration.ParagraphActionsIT testDisa

[GitHub] zeppelin issue #1951: [ZEPPELIN-2020] Invoke method from resource

2017-01-29 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1951 Merge to master if no further discussions --- 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 feat

[GitHub] zeppelin issue #1663: ZEPPELIN-1692: Ability to access Spark jobs UI from th...

2017-01-29 Thread karuppayya
Github user karuppayya commented on the issue: https://github.com/apache/zeppelin/pull/1663 Thanks @felixcheung . Looks like i read only part of the exception line and thought it was not due to this PR. I have fixed it now. The exception related to `websocket connected` tooltip sti

[GitHub] zeppelin pull request #1957: Branch 0.6 Groovy Interpreter

2017-01-29 Thread dlukyanov
GitHub user dlukyanov opened a pull request: https://github.com/apache/zeppelin/pull/1957 Branch 0.6 Groovy Interpreter ### What is this PR for? Groovy Interpreter ### What type of PR is it? Feature ### Todos * [Code] - Task * [Samples] - Task *