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

[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

[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

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

[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 #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 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 #955: [ZEPPELIN-940] Allow zeppelin server to conne...

2016-06-07 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 #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 does

[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 project is set

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

[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

[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

[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

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

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

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

2016-06-08 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 email

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

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

2016-06-08 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 merged

[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

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

[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

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

[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 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] zeppelin issue #1000: typo

2016-06-12 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1000 @OutOfBedlam Thanks for the fix. LGTM. The failure of CI looks irrelevant. --- 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] zeppelin pull request #990: [MINOR] Removed classloaderInterpreter on Remote...

2016-06-10 Thread jongyoul
GitHub user jongyoul opened a pull request: https://github.com/apache/zeppelin/pull/990 [MINOR] Removed classloaderInterpreter on RemoteInterpreterServer ### What is this PR for? Removed usage of ClassloaderInterpreter while RemoteInterpreter is being created. ### What

[GitHub] zeppelin issue #984: [ZEPPELIN-982] Improve interpreter completion API

2016-06-10 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/984 @AhyoungRyu What the `value` of InterpreterCompletion means? And I suggest we remove the mark abstract from Interpreter class because almost interpreter returns empty list and it's better

[GitHub] zeppelin pull request #990: [MINOR] Removed classloaderInterpreter on Remote...

2016-06-11 Thread jongyoul
GitHub user jongyoul reopened a pull request: https://github.com/apache/zeppelin/pull/990 [MINOR] Removed classloaderInterpreter on RemoteInterpreterServer ### What is this PR for? Removed usage of ClassloaderInterpreter while RemoteInterpreter is being created

[GitHub] zeppelin pull request #990: [MINOR] Removed classloaderInterpreter on Remote...

2016-06-11 Thread jongyoul
Github user jongyoul closed the pull request at: https://github.com/apache/zeppelin/pull/990 --- 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

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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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

[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 LGTM except minor 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 enabled

[GitHub] zeppelin pull request #987: ZEPPELIN-694 Remove ClassLoaderInterpreter

2016-06-10 Thread jongyoul
GitHub user jongyoul opened a pull request: https://github.com/apache/zeppelin/pull/987 ZEPPELIN-694 Remove ClassLoaderInterpreter ### What is this PR for? Removing some redundant codes for simplifying Zeppelin. ### What type of PR is it? [Refactoring

[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 @SachinJanani Yes, it's one of the best place. --- 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] zeppelin issue #806: [ZEPPELIN-707]Automatically adds %.* of previous paragr...

2016-06-10 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/806 It works well with another default interpreter except `%spark`. See the below. https://cloud.githubusercontent.com/assets/3612566/15959490/0720f9e6-2f36-11e6-9f8b-ad6f627c4582.png;> ht

[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

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

[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

[GitHub] zeppelin issue #990: [MINOR] Removed classloaderInterpreter on RemoteInterpr...

2016-06-11 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/990 @Leemoonsoo Review this, please. --- 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] zeppelin pull request #982: ZEPPELIN-934 Merge Phoenix interpreter into JDBC...

2016-06-11 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 #985: ZEPPELIN-974 Merge TajoInterpreter into JdbcInte...

2016-06-11 Thread jongyoul
Github user jongyoul closed the pull request at: https://github.com/apache/zeppelin/pull/985 --- 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

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

2016-06-11 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

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

2016-06-11 Thread jongyoul
GitHub user jongyoul reopened 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

[GitHub] zeppelin issue #972: ZEPPELIN-963 ] Jobmanagement - (1) basic backend.

2016-06-13 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/972 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 does not have

[GitHub] zeppelin issue #1002: ZEPPELIN-992

2016-06-13 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1002 The failures is related to ZEPPELIN-878 --- 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 #1002: ZEPPELIN-992

2016-06-13 Thread jongyoul
GitHub user jongyoul reopened a pull request: https://github.com/apache/zeppelin/pull/1002 ZEPPELIN-992 ### What is this PR for? Moving Interpreter.interpret into LazyOpenInterpreterTest in oder to divide test scope between InterpreterFactoryTest and LazyOpenInterpreter

[GitHub] zeppelin pull request #1002: ZEPPELIN-992

2016-06-13 Thread jongyoul
GitHub user jongyoul opened a pull request: https://github.com/apache/zeppelin/pull/1002 ZEPPELIN-992 ### What is this PR for? Moving Interpreter.interpret into LazyOpenInterpreterTest in oder to divide test scope between InterpreterFactoryTest and LazyOpenInterpreter

[GitHub] zeppelin pull request #1002: ZEPPELIN-992

2016-06-13 Thread jongyoul
Github user jongyoul closed the pull request at: https://github.com/apache/zeppelin/pull/1002 --- 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

[GitHub] zeppelin issue #990: [MINOR] Removed classloaderInterpreter on RemoteInterpr...

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

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

2016-06-11 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

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

2016-06-11 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

[GitHub] zeppelin issue #984: [ZEPPELIN-982] Improve interpreter completion API

2016-06-11 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/984 @AhyoungRyu I see. 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] zeppelin pull request #974: ZEPPELIN-614 Part 1. Merge PostgresqlInterpreter...

2016-06-11 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

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

2016-06-11 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

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

2016-06-13 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

[GitHub] zeppelin pull request #1005: ZEPPELIN-995 Change scheduler for JDBC interpre...

2016-06-13 Thread jongyoul
Github user jongyoul closed the pull request at: https://github.com/apache/zeppelin/pull/1005 --- 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

[GitHub] zeppelin pull request #1005: ZEPPELIN-995 Change scheduler for JDBC interpre...

2016-06-13 Thread jongyoul
GitHub user jongyoul reopened a pull request: https://github.com/apache/zeppelin/pull/1005 ZEPPELIN-995 Change scheduler for JDBC interpreter to use concurrent execution ### What is this PR for? Changed scheduler from FIFO to Parallels in JdbcInterpreter. This is a default

[GitHub] zeppelin issue #1005: ZEPPELIN-995 Change scheduler for JDBC interpreter to ...

2016-06-14 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1005 @Leemoonsoo I also agree to makes this configurable. I'll follow up this with another PR. I also think we need to replace using getScheduler to another way. --- If your project is set up

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

2016-06-14 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-14 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

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

2016-06-13 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 #1005: ZEPPELIN-995 Change scheduler for JDBC interpre...

2016-06-13 Thread jongyoul
GitHub user jongyoul opened a pull request: https://github.com/apache/zeppelin/pull/1005 ZEPPELIN-995 Change scheduler for JDBC interpreter to use concurrent execution ### What is this PR for? Changed scheduler from FIFO to Parallels in JdbcInterpreter. This is a default

[GitHub] zeppelin issue #1002: ZEPPELIN-992

2016-06-13 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1002 @bzz Thanks for the guide. I'm good at selenium, thus I thought ZeppelinIT.testSparkInterpreterDependencyLoading was related to that issue. I'll look into it but do you know how to reproduce

[GitHub] zeppelin issue #1011: Update licenses

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

[GitHub] zeppelin pull request #1012: ZEPPELIN-999 Support alias for JDBC properties

2016-06-14 Thread jongyoul
GitHub user jongyoul opened a pull request: https://github.com/apache/zeppelin/pull/1012 ZEPPELIN-999 Support alias for JDBC properties ### What is this PR for? In case of using JdbcInterpreter, you should use %jdbc(prefix) if you set multiple configurations. This PR makes you

[GitHub] zeppelin issue #1005: ZEPPELIN-995 Change scheduler for JDBC interpreter to ...

2016-06-14 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1005 @Leemoonsoo I've missed something. I've known it was not configurable dynamically. Let me check. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] zeppelin issue #1012: ZEPPELIN-999 Support alias for JDBC properties

2016-06-14 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1012 @felixcheung I also have a plan to generalize this feature after 0.6.0 releases. :-) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] zeppelin issue #1005: ZEPPELIN-995 Change scheduler for JDBC interpreter to ...

2016-06-15 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1005 @Leemoonsoo I've totally understood your idea and agree with you. I'll patch for it and push it again --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] zeppelin issue #1012: ZEPPELIN-999 Support alias for JDBC properties

2016-06-15 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1012 @bzz I'll merge it into master and branch-0.6 --- 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] zeppelin issue #972: ZEPPELIN-963 ] Jobmanagement - (1) basic backend.

2016-06-15 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/972 Merging it into master branch only. --- 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 #982: ZEPPELIN-934 Merge Phoenix interpreter into JDBC...

2016-06-15 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

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

2016-06-15 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 #1002: ZEPPELIN-992

2016-06-13 Thread jongyoul
Github user jongyoul closed the pull request at: https://github.com/apache/zeppelin/pull/1002 --- 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

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

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

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

[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

[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: [MINOR] fixed a logic to find iss...

2016-05-31 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: [DOC] Added the link of Apache's ...

2016-05-31 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 project

[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

  1   2   3   4   5   6   7   8   9   10   >