[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 #996: [ZEPPELIN-890] Cache downloaded file from 'maven...

2016-06-11 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/996 --- 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 #996: [ZEPPELIN-890] Cache downloaded file from 'maven-downlo...

2016-06-11 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/996 Sure, I think we can merge this as a hotfix. --- 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 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 by the

[GitHub] zeppelin issue #996: [ZEPPELIN-890] Cache downloaded file from 'maven-downlo...

2016-06-11 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/996 @prabhjyotsingh Thanks for suggestion. Once travis caches '.m2/repository/.cache/maven-download-plugin' directory, 'maven-download-plugin' will not even try to download files from internet. So

[GitHub] zeppelin issue #965: [ZEPPELIN-957] Normalize note name.

2016-06-11 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/965 @astroshim Thanks! `MyNote` and `/MyNote` are treated differently. And i think it's right. (compatible to previous version) Therefore I think it's more consistent to treat differently

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

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

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

[GitHub] zeppelin issue #965: [ZEPPELIN-957] Normalize note name.

2016-06-11 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/965 I really appreciate your detailed review. @Leemoonsoo I will fix 1,2,3 but in case 4, I thought ```/MyNote/sub``` and ```MyNote/sub``` are the same from directory point of view. Do

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

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