[GitHub] incubator-zeppelin pull request: [MINOR] Remove redundant logic of...

2016-05-22 Thread jongyoul
GitHub user jongyoul opened a pull request: https://github.com/apache/incubator-zeppelin/pull/909 [MINOR] Remove redundant logic of JDBC Interpreter ### What is this PR for? Removing redundant codes and expanding features for passing extra properties into JDBC driver

[GitHub] incubator-zeppelin pull request: [MINOR] Remove redundant logic of...

2016-05-22 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/909#issuecomment-220835870 @wfhartford I think it's okay to satisfy your concern. Could you please review it? --- If your project is set up for it, you can reply to this emai

[GitHub] incubator-zeppelin pull request: [MINOR] Remove redundant logic of...

2016-05-22 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/909#issuecomment-220872955 This PR is a very trivial. Merging if there's no more discussion. --- If your project is set up for it, you can reply to this email and have your

[GitHub] incubator-zeppelin pull request: ZEPPELIN-804 Refactoring registra...

2016-05-24 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/835#issuecomment-221451567 @echarles Hi, I have a problem for testing SparkRInterpreter of what you code. Could you please help solve this issue below? ``` Results

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-881] Apache Ignite vers...

2016-05-25 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/916#issuecomment-221767576 Simple! Great! And I think Ignite JDBC interpreter doesn't have any special thing. How do you think it merges into JDBC interpreter? --- If your pr

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-881] Apache Ignite vers...

2016-05-25 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/916#issuecomment-221767639 Without the issue above, this PR looks good to me. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] incubator-zeppelin pull request: ZEPPELIN-804 Refactoring registra...

2016-05-26 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/835#issuecomment-221816292 @echarles Thanks for the advices and I've found the reason why zeppelin.R.render.options. This is because ```-Pr -Pspark```. According to thi

[GitHub] incubator-zeppelin pull request: ZEPPELIN-804 Refactoring registra...

2016-05-26 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/835#issuecomment-221817376 @echarles You're right but we need to pass the CI first. :-) I don't think this is a real case and I'll handle it through another PR whi

[GitHub] incubator-zeppelin pull request: ZEPPELIN-804 Refactoring registra...

2016-05-26 Thread jongyoul
GitHub user jongyoul reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/835 ZEPPELIN-804 Refactoring registration mechanism on Interpreters ### What is this PR for? This PR enable Zeppelin server register Interpreters without any dependencies of their

[GitHub] incubator-zeppelin pull request: ZEPPELIN-804 Refactoring registra...

2016-05-26 Thread jongyoul
Github user jongyoul closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/835 --- 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] incubator-zeppelin pull request: ZEPPELIN-804 Refactoring registra...

2016-05-26 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/835#issuecomment-221870018 re-trigger --- 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

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-881] Apache Ignite vers...

2016-05-26 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/916#issuecomment-221882257 @agura I'll thank you if you help merge Ignite JDBC into generic JDBC interpreter. And about this PR, it looks good to me. Merging if there

[GitHub] incubator-zeppelin pull request: ZEPPELIN-804 Refactoring registra...

2016-05-26 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/835#issuecomment-221884372 Finally, I've passed CI. Summary of changes, - Read interpreter info from file with json format - Fixed some tests in order to pass wi

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-870] Improvement build ...

2016-05-26 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/905#issuecomment-221920236 @shijinkui It becomes green. Could you please trigger it again? --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-870] Improvement build ...

2016-05-26 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/905#issuecomment-222056071 @shijinkui Just close and reopen 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

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-870] Improvement build ...

2016-05-27 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/905#issuecomment-222162047 I agree with @bzz. If your PR has both of parts which change behavior or not, I suggest you divide two seperate PR because of easy review. If you mix them

[GitHub] incubator-zeppelin pull request: ZEPPELIN-804 Refactoring registra...

2016-05-28 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/835#issuecomment-222346004 @Leemoonsoo Sure. I'll update 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

[GitHub] incubator-zeppelin pull request: ZEPPELIN-804 Refactoring registra...

2016-05-29 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/835#issuecomment-222369623 @bzz In my opinion, there's no side-effects of this feature. --- If your project is set up for it, you can reply to this email and have your reply a

[GitHub] incubator-zeppelin pull request: ZEPPELIN-804 Refactoring registra...

2016-05-29 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/835#issuecomment-222416923 Merging if there's no more discussion. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as wel

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-881] Apache Ignite vers...

2016-05-30 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/916#issuecomment-222491988 @agura Hi, one of CI test is failed. Could you please check it? --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-881] Apache Ignite vers...

2016-05-30 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/916#issuecomment-222493080 @agura Close and re-open this PR --- 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

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-881] Apache Ignite vers...

2016-05-30 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/916#issuecomment-222504357 @Leemoonsoo Could you please check that failure due to flaky test? --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] incubator-zeppelin pull request: [ZEPPELIN 894] add new hadoop/spa...

2016-05-30 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/923#issuecomment-222579535 I agree to change the default version of Spark. LGTM. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] incubator-zeppelin pull request: Remove unnedeed commons-exec decl...

2016-05-30 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/931#issuecomment-222580712 LGTM. Merging if there's no more discussion --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] incubator-zeppelin pull request: [DOC] Added the link of Apache's ...

2016-05-30 Thread jongyoul
GitHub user jongyoul opened a pull request: https://github.com/apache/incubator-zeppelin/pull/934 [DOC] Added the link of Apache's Jira for beginner ### What is this PR for? Added documentation for new contributors ### What type of PR is it? [Document

[GitHub] incubator-zeppelin pull request: Pool pluggable rebase

2016-05-30 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/843#issuecomment-222591848 @Leemoonsoo @felixcheung Ping. --- 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

[GitHub] incubator-zeppelin pull request: [DOC] Added the link of Apache's ...

2016-05-30 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/934#issuecomment-222600583 I'll merge it in a minute. --- 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 pr

[GitHub] incubator-zeppelin pull request: [MINOR] fixed a logic to find iss...

2016-05-30 Thread jongyoul
GitHub user jongyoul opened a pull request: https://github.com/apache/incubator-zeppelin/pull/936 [MINOR] fixed a logic to find issue number from title ### What is this PR for? Changed the regex to find a issue number from title ### What type of PR is it? [Bug Fix

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-881] Apache Ignite version update...

2016-05-31 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/916 We also appreciate your contribution! --- 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

[GitHub] incubator-zeppelin pull request: [MINOR] fixed a logic to find issue number ...

2016-05-31 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/936 Merging if there's no more discussion. --- 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 doe

[GitHub] incubator-zeppelin pull request: Remove incubating from pom files

2016-05-31 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/942 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] incubator-zeppelin pull request: ZEPPELIN-598 ] Dynamic loading for Interpre...

2016-05-31 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/908 Ping. I make a new issue that may break this PR - https://issues.apache.org/jira/browse/ZEPPELIN-932 -. Could anyone please check the status of this PR? --- If your project is

[GitHub] incubator-zeppelin pull request: ZEPPELIN-925 Merge HiveInterpreter into JDB...

2016-05-31 Thread jongyoul
GitHub user jongyoul opened a pull request: https://github.com/apache/incubator-zeppelin/pull/943 ZEPPELIN-925 Merge HiveInterpreter into JDBCInterpreter ### What is this PR for? A few sentences describing the overall goals of the pull request's commits. First time?

[GitHub] incubator-zeppelin pull request: ZEPPELIN-925 Merge HiveInterpreter into JDB...

2016-06-01 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/943 @minahlee Could you please review this PR, especially setting dependencies? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] incubator-zeppelin pull request: Remove incubating from doc

2016-06-01 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/941 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] incubator-zeppelin pull request: ZEPPELIN-925 Merge HiveInterpreter into JDB...

2016-06-01 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/943 That failure is irrelevant to this PR. ``` Results : Failed tests: ZeppelinSparkClusterTest.pySparkDepLoaderTest:231 expected: but was: Tests run

[GitHub] incubator-zeppelin pull request: ZEPPELIN-925 Merge HiveInterpreter into JDB...

2016-06-01 Thread jongyoul
Github user jongyoul closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/943 --- 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] incubator-zeppelin pull request: ZEPPELIN-925 Merge HiveInterpreter into JDB...

2016-06-01 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/943 re-trigger --- 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

[GitHub] incubator-zeppelin pull request: ZEPPELIN-925 Merge HiveInterpreter into JDB...

2016-06-01 Thread jongyoul
GitHub user jongyoul reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/943 ZEPPELIN-925 Merge HiveInterpreter into JDBCInterpreter ### What is this PR for? This removes hive module and adds example setting for using Hive in a JdbcInterpreter by using

[GitHub] incubator-zeppelin issue #943: ZEPPELIN-925 Merge HiveInterpreter into JDBCI...

2016-06-01 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/943 @bzz What you have seen is different from the first error. Have you ever seen the first one before? I'll make a new issue for second error with `flaky-test` tag. And it&#

[GitHub] incubator-zeppelin issue #943: ZEPPELIN-925 Merge HiveInterpreter into JDBCI...

2016-06-01 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/943 @bzz I made a https://issues.apache.org/jira/browse/ZEPPELIN-936 --- 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

[GitHub] incubator-zeppelin issue #945: ZEPPELIN-931: fix interpreter listing bug

2016-06-01 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/945 @AhyoungRyu I've tested and reproduced what you mentioned. Your change looks simple but it's not perfect because you cannot see spark interpreter in your interpreter li

[GitHub] incubator-zeppelin issue #945: ZEPPELIN-931: fix interpreter listing bug

2016-06-01 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/945 I don't know whether front-end uses RegisteredInterprerter. We need to find a way to indicate or mark which class is used in front and backend. --- If your project is set up f

[GitHub] incubator-zeppelin issue #945: ZEPPELIN-931: fix interpreter listing bug

2016-06-01 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/945 @AhyoungRyu Could you please add "[HOTFIX]" in a title? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If yo

[GitHub] incubator-zeppelin issue #943: ZEPPELIN-925 Merge HiveInterpreter into JDBCI...

2016-06-01 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/943 @bzz @minahlee Please check it finally. And I email {dev,user}@ to announce this change. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] incubator-zeppelin issue #945: [HOTFIX] ZEPPELIN-931: fix interpreter listin...

2016-06-01 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/945 @AhyoungRyu I've missed something. I submitted a PR, 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

[GitHub] incubator-zeppelin issue #945: [HOTFIX] ZEPPELIN-931: fix interpreter listin...

2016-06-01 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/945 @AhyoungRyu Thanks. --- 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

[GitHub] incubator-zeppelin issue #945: [HOTFIX] ZEPPELIN-931: fix interpreter listin...

2016-06-01 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/945 LGTM. I'll merge it asap. --- 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 fe

[GitHub] incubator-zeppelin issue #943: ZEPPELIN-925 Merge HiveInterpreter into JDBCI...

2016-06-01 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/943 @felixcheung How about 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

[GitHub] incubator-zeppelin issue #943: ZEPPELIN-925 Merge HiveInterpreter into JDBCI...

2016-06-02 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/943 re-trigger --- 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] incubator-zeppelin pull request #943: ZEPPELIN-925 Merge HiveInterpreter int...

2016-06-02 Thread jongyoul
Github user jongyoul closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/943 --- 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] incubator-zeppelin pull request #943: ZEPPELIN-925 Merge HiveInterpreter int...

2016-06-02 Thread jongyoul
GitHub user jongyoul reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/943 ZEPPELIN-925 Merge HiveInterpreter into JDBCInterpreter ### What is this PR for? This removes hive module and adds example setting for using Hive in a JdbcInterpreter by using

[GitHub] incubator-zeppelin issue #954: [ZEPPELIN-915] New registration mechanism app...

2016-06-02 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/954 @ralphge Thanks for the contribution. I've tested and worked well. And CI failure is a temporary issue. Could you please re-trigger CI? It's OK to close and re-open this PR

[GitHub] incubator-zeppelin issue #954: [ZEPPELIN-915] New registration mechanism app...

2016-06-02 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/954 In addition to step for test, you should remove conf/interpreter.json before you start. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] incubator-zeppelin issue #956: [ZEPPELIN-923] Apply new mechanism to ShellIn...

2016-06-03 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/956 😄 --- 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

[GitHub] incubator-zeppelin issue #956: [ZEPPELIN-923] Apply new mechanism to ShellIn...

2016-06-03 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/956 @AhyoungRyu One looks irrelevant and another is about rat plugin. you need to exclude interpreter-setting.json in a rat plugin. --- If your project is set up for it, you can reply to

[GitHub] incubator-zeppelin pull request #957: ZEPPELIN-900 Show interpreter logs in ...

2016-06-03 Thread jongyoul
GitHub user jongyoul opened a pull request: https://github.com/apache/incubator-zeppelin/pull/957 ZEPPELIN-900 Show interpreter logs in front side's result area while processing ### What is this PR for? Showing logs while running paragraph. This is POC leve

[GitHub] incubator-zeppelin issue #954: [ZEPPELIN-915] New registration mechanism app...

2016-06-06 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/954 LGTM. Merging if there's no more discussion. --- 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 doe

[GitHub] incubator-zeppelin issue #957: ZEPPELIN-900 Show interpreter logs in front s...

2016-06-06 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/957 @bzz Thanks for the review. Quick answer for last question. InterpreterResult include new logs only after `getProgress()` called before. See [here](https://github.com/apache/incubator

[GitHub] incubator-zeppelin issue #955: [ZEPPELIN-940] Allow zeppelin server to conne...

2016-06-06 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/955 @SachinJanani Thanks for the contribution. BTW, could you please rebase your change from master? If you merge something from master, it makes review on your PR hard. --- If your

[GitHub] incubator-zeppelin issue #943: ZEPPELIN-925 Merge HiveInterpreter into JDBCI...

2016-06-06 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/943 Update documentation and rebase it from current master --- 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

[GitHub] incubator-zeppelin issue #972: ZEPPELIN-531 ] Jobmanagement - (1) basic back...

2016-06-07 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/972 Thanks. Do you have any test scenario? It would help to add test cases. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] incubator-zeppelin issue #970: Update README.md for fixing ignite flag

2016-06-07 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/970 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

[GitHub] incubator-zeppelin issue #966: [ZEPPELIN-958] Support syntax highlight for p...

2016-06-07 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/966 @felixcheung Could you please tell me your idea in details? Basically I agree with you, but I'm not good at frontend, I don't know whether the feature can be adopted dynamica

[GitHub] incubator-zeppelin issue #966: [ZEPPELIN-958] Support syntax highlight for p...

2016-06-07 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/966 BTW, this PR looks good to me. We can discuss @felixcheung's issue after merging this PR. --- If your project is set up for it, you can reply to this email and have your reply a

[GitHub] incubator-zeppelin issue #954: [ZEPPELIN-915] New registration mechanism app...

2016-06-07 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/954 @ralphge Could you please tell me your jira id? I cannot find you on jira with your github id. https://issues.apache.org/jira/browse/ZEPPELIN-915 --- If your project is set up

[GitHub] incubator-zeppelin pull request #943: ZEPPELIN-925 Merge HiveInterpreter int...

2016-06-07 Thread jongyoul
Github user jongyoul closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/943 --- 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] incubator-zeppelin pull request #943: ZEPPELIN-925 Merge HiveInterpreter int...

2016-06-07 Thread jongyoul
GitHub user jongyoul reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/943 ZEPPELIN-925 Merge HiveInterpreter into JDBCInterpreter ### What is this PR for? This removes hive module and adds example setting for using Hive in a JdbcInterpreter by using

[GitHub] incubator-zeppelin issue #972: ZEPPELIN-531 ] Jobmanagement - (1) basic back...

2016-06-07 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/972 How about changing this title to add ZEPPELIN-963? --- 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

[GitHub] incubator-zeppelin issue #943: ZEPPELIN-925 Merge HiveInterpreter into JDBCI...

2016-06-07 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/943 @felixcheung @bzz @minahlee @AhyoungRyu I've reverted hive.md and updated documentation. Please check it finally. --- If your project is set up for it, you can reply to this emai

[GitHub] incubator-zeppelin pull request #974: ZEPPELIN-614 Part 1. Merge PostgresqlI...

2016-06-07 Thread jongyoul
GitHub user jongyoul opened a pull request: https://github.com/apache/incubator-zeppelin/pull/974 ZEPPELIN-614 Part 1. Merge PostgresqlInterpreter's sql completer into JDBC Interpreter ### What is this PR for? Supporting Sql completer into JDBC Interpreter which is merg

[GitHub] incubator-zeppelin issue #974: ZEPPELIN-614 Part 1. Merge PostgresqlInterpre...

2016-06-07 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/974 @tzolov Could you please test this PR with Postrgres, HAWQ and so on? I've heard that you are one of the heavy users. --- If your project is set up for it, you can reply to this

[GitHub] incubator-zeppelin issue #943: ZEPPELIN-925 Merge HiveInterpreter into JDBCI...

2016-06-08 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/943 @vgmartinez Now, Zeppelin enable load dependencies dynamically, thus if you set dependencies in your interpreter tab, it works. Thus, we don't have to include any dependencies any

[GitHub] incubator-zeppelin pull request #974: ZEPPELIN-614 Part 1. Merge PostgresqlI...

2016-06-08 Thread jongyoul
Github user jongyoul closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/974 --- 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] incubator-zeppelin pull request #974: ZEPPELIN-614 Part 1. Merge PostgresqlI...

2016-06-08 Thread jongyoul
GitHub user jongyoul reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/974 ZEPPELIN-614 Part 1. Merge PostgresqlInterpreter's sql completer into JDBC Interpreter ### What is this PR for? Supporting Sql completer into JDBC Interpreter which is merg

[GitHub] incubator-zeppelin issue #943: ZEPPELIN-925 Merge HiveInterpreter into JDBCI...

2016-06-08 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/943 Merging there's no more discussion --- 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

[GitHub] incubator-zeppelin issue #955: [ZEPPELIN-940] Allow zeppelin server to conne...

2016-06-08 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/955 That failure is related to https://issues.apache.org/jira/browse/ZEPPELIN-878, but I have a question. What kind of procedure do you do while you run Zeppelin? Generally, when you stop

[GitHub] incubator-zeppelin issue #955: [ZEPPELIN-940] Allow zeppelin server to conne...

2016-06-08 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/955 @SachinJanani Thanks for sharing. I also think we should consider running Zeppelin's interpreters in a different machine. This would be a starting point to do this. --- If

[GitHub] incubator-zeppelin issue #977: [ZEPPELIN-911] Apply new mechanism to FlinkIn...

2016-06-09 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/977 I've tested it and worked well. LGTM. Merging if there's no more discussion. @rerngvit Could you please your jira id? I cannot assign you as a contributor --- If your proj

[GitHub] incubator-zeppelin issue #955: [ZEPPELIN-940] Allow zeppelin server to conne...

2016-06-09 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/955 @Leemoonsoo Who's proper reviewer for UI part? --- 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 doe

[GitHub] incubator-zeppelin issue #974: ZEPPELIN-614 Part 1. Merge PostgresqlInterpre...

2016-06-09 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/974 @bzz I updated license files. could you please check it again? https://github.com/apache/incubator-zeppelin/pull/974/commits/99ce452b68c757c1959b9d4b7983f7d81000256e --- If

[GitHub] incubator-zeppelin pull request #981: ZEPPELIN-925 Part 2. Merge Hive Interp...

2016-06-09 Thread jongyoul
GitHub user jongyoul opened a pull request: https://github.com/apache/incubator-zeppelin/pull/981 ZEPPELIN-925 Part 2. Merge Hive Interpreter into JDBC ### What is this PR for? Fixed some wrong documentation and settings. This is minor issue. ### What type of PR is it

[GitHub] incubator-zeppelin pull request #974: ZEPPELIN-614 Part 1. Merge PostgresqlI...

2016-06-09 Thread jongyoul
Github user jongyoul closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/974 --- 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] incubator-zeppelin pull request #974: ZEPPELIN-614 Part 1. Merge PostgresqlI...

2016-06-09 Thread jongyoul
GitHub user jongyoul reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/974 ZEPPELIN-614 Part 1. Merge PostgresqlInterpreter's sql completer into JDBC Interpreter ### What is this PR for? Supporting Sql completer into JDBC Interpreter which is merg

[GitHub] incubator-zeppelin pull request #982: ZEPPELIN-934 Merge Phoenix interpreter...

2016-06-09 Thread jongyoul
GitHub user jongyoul opened a pull request: https://github.com/apache/incubator-zeppelin/pull/982 ZEPPELIN-934 Merge Phoenix interpreter into JDBC interpreter ### What is this PR for? Removing Phoenix interpreter and adding example for using it through JDBC ### What

[GitHub] incubator-zeppelin pull request #974: ZEPPELIN-614 Part 1. Merge PostgresqlI...

2016-06-09 Thread jongyoul
Github user jongyoul closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/974 --- 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] incubator-zeppelin pull request #974: ZEPPELIN-614 Part 1. Merge PostgresqlI...

2016-06-09 Thread jongyoul
GitHub user jongyoul reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/974 ZEPPELIN-614 Part 1. Merge PostgresqlInterpreter's sql completer into JDBC Interpreter ### What is this PR for? Supporting Sql completer into JDBC Interpreter which is merg

[GitHub] incubator-zeppelin issue #974: ZEPPELIN-614 Part 1. Merge PostgresqlInterpre...

2016-06-09 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/974 npm build error --- 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

[GitHub] zeppelin issue #955: [ZEPPELIN-940] Allow zeppelin server to connect to alre...

2016-06-09 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/955 @Leemoonsoo Yes, I just think there're too many requests for him, so I'm asking you to have another candidate. --- If your project is set up for it, you can reply to this email and

[GitHub] zeppelin pull request #974: ZEPPELIN-614 Part 1. Merge PostgresqlInterpreter...

2016-06-09 Thread jongyoul
GitHub user jongyoul reopened a pull request: https://github.com/apache/zeppelin/pull/974 ZEPPELIN-614 Part 1. Merge PostgresqlInterpreter's sql completer into JDBC Interpreter ### What is this PR for? Supporting Sql completer into JDBC Interpreter which is merged b

[GitHub] zeppelin pull request #974: ZEPPELIN-614 Part 1. Merge PostgresqlInterpreter...

2016-06-09 Thread jongyoul
Github user jongyoul closed the pull request at: https://github.com/apache/zeppelin/pull/974 --- 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 #981: ZEPPELIN-925 Part 2. Merge Hive Interpreter into...

2016-06-09 Thread jongyoul
GitHub user jongyoul reopened a pull request: https://github.com/apache/zeppelin/pull/981 ZEPPELIN-925 Part 2. Merge Hive Interpreter into JDBC ### What is this PR for? Fixed some wrong documentation and settings. This is minor issue. ### What type of PR is it

[GitHub] zeppelin pull request #981: ZEPPELIN-925 Part 2. Merge Hive Interpreter into...

2016-06-09 Thread jongyoul
Github user jongyoul closed the pull request at: https://github.com/apache/zeppelin/pull/981 --- 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 #982: ZEPPELIN-934 Merge Phoenix interpreter into JDBC...

2016-06-09 Thread jongyoul
GitHub user jongyoul reopened a pull request: https://github.com/apache/zeppelin/pull/982 ZEPPELIN-934 Merge Phoenix interpreter into JDBC interpreter ### What is this PR for? Removing Phoenix interpreter and adding example for using it through JDBC ### What type of PR

[GitHub] zeppelin pull request #982: ZEPPELIN-934 Merge Phoenix interpreter into JDBC...

2016-06-09 Thread jongyoul
Github user jongyoul closed the pull request at: https://github.com/apache/zeppelin/pull/982 --- 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 #955: [ZEPPELIN-940] Allow zeppelin server to connect to alre...

2016-06-09 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/955 @corneadoug Thanks, and about review, I have been worried that you have been pressed with all of review requests of front-side issues. --- If your project is set up for it, you can reply to this

[GitHub] zeppelin pull request #985: ZEPPELIN-974 Merge TajoInterpreter into JdbcInte...

2016-06-09 Thread jongyoul
GitHub user jongyoul opened a pull request: https://github.com/apache/zeppelin/pull/985 ZEPPELIN-974 Merge TajoInterpreter into JdbcInterpreter ### What is this PR for? Merging TajoInterpreter into JdbcInterpreter, and removing TajoInterprete ### What type of PR is

[GitHub] zeppelin pull request #974: ZEPPELIN-614 Part 1. Merge PostgresqlInterpreter...

2016-06-09 Thread jongyoul
GitHub user jongyoul reopened a pull request: https://github.com/apache/zeppelin/pull/974 ZEPPELIN-614 Part 1. Merge PostgresqlInterpreter's sql completer into JDBC Interpreter ### What is this PR for? Supporting Sql completer into JDBC Interpreter which is merged b

[GitHub] zeppelin pull request #974: ZEPPELIN-614 Part 1. Merge PostgresqlInterpreter...

2016-06-09 Thread jongyoul
Github user jongyoul closed the pull request at: https://github.com/apache/zeppelin/pull/974 --- 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 #981: ZEPPELIN-925 Part 2. Merge Hive Interpreter into...

2016-06-09 Thread jongyoul
GitHub user jongyoul reopened a pull request: https://github.com/apache/zeppelin/pull/981 ZEPPELIN-925 Part 2. Merge Hive Interpreter into JDBC ### What is this PR for? Fixed some wrong documentation and settings. This is minor issue. ### What type of PR is it

  1   2   3   4   5   6   7   8   9   10   >