[GitHub] zeppelin issue #1284: [ZEPPELIN-1291]Tagging meta value on interpreter compl...

2016-08-07 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1284 @felixcheung Thank you for your review. I think The meta name of auto completion should be original meta name because the user interpreters name that you mentioned(spark1,spark2) is not

[GitHub] zeppelin issue #1284: [ZEPPELIN-1291]Tagging meta value on interpreter compl...

2016-08-08 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1284 Even though the interpreter name is ```spark1``` or ```myintp```, the auto completion is spark's. so This way(just spark) is clear i think. --- If your project is set up for it, yo

[GitHub] zeppelin pull request #1302: [ZEPPELIN-1306] Interpreter restarts on a note.

2016-08-08 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1302 [ZEPPELIN-1306] Interpreter restarts on a note. ### What is this PR for? This PR is for usability of restarting interpreter. ### What type of PR is it? Improvement

[GitHub] zeppelin issue #1229: [BugFix] Show checkbox for "Connect to existing proces...

2016-08-08 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1229 okay I'll make 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 w

[GitHub] zeppelin issue #1257: [Zeppelin-945] Interpreter authorization

2016-08-08 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1257 @prabhjyotsingh Thank you for your detail review. I fixed all that you mentioned. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] zeppelin pull request #1318: [ZEPPELIN-1280][Spark on Yarn] Documents for ru...

2016-08-10 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1318 [ZEPPELIN-1280][Spark on Yarn] Documents for running zeppelin on production environments. ### What is this PR for? This PR is for the documentation of running zeppelin on production

[GitHub] zeppelin issue #1318: [ZEPPELIN-1280][Spark on Yarn] Documents for running z...

2016-08-10 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1318 @AhyoungRyu I'll update. Thank you for giving me the informations. --- 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] zeppelin issue #1318: [ZEPPELIN-1280][Spark on Yarn] Documents for running z...

2016-08-11 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1318 @felixcheung Thank you very much for detail review.:+ I'll fix them but I wonder if what spark and hadoop version should be supported in this document. --- If your project is set up f

[GitHub] zeppelin issue #1318: [ZEPPELIN-1280][Spark on Yarn] Documents for running z...

2016-08-11 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1318 Do you know what version of spark&hadoop is popular? I will test 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] zeppelin issue #1318: [ZEPPELIN-1280][Spark on Yarn] Documents for running z...

2016-08-11 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1318 Spark2.0 & hadoop2.3 is working well. ![image](https://cloud.githubusercontent.com/assets/3348133/17588549/d611d7ca-600a-11e6-87f6-4a692fbae273.png) --- If your project is set up fo

[GitHub] zeppelin issue #1193: [ZEPPELIN-1192] Block pyspark paragraph hang.

2016-08-11 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1193 @bzz I added log and tested. Thank you for taking care of this. --- 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] zeppelin issue #1318: [ZEPPELIN-1280][Spark on Yarn] Documents for running z...

2016-08-12 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1318 Then what about supporting latest verison(Spark2.0 & hadoop2.7)? --- 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 pro

[GitHub] zeppelin issue #1318: [ZEPPELIN-1280][Spark on Yarn] Documents for running z...

2016-08-12 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1318 Spark2.0 & hadoop2.7 is working well. - YARN ![image](https://cloud.githubusercontent.com/assets/3348133/17622873/fc1b827c-60d7-11e6-80ec-e8e9890a5c99.png) - hdfs ![i

[GitHub] zeppelin issue #1193: [ZEPPELIN-1192] Block pyspark paragraph hang.

2016-08-12 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1193 @bzz I rebased. 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 #1257: [Zeppelin-945] Interpreter authorization

2016-08-15 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1257 @bzz please review and Thanks 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 your project does not have this feature

[GitHub] zeppelin issue #1257: [Zeppelin-945] Interpreter authorization

2016-08-15 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1257 @bzz I just fixed, I missed that. --- 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 issue #1193: [ZEPPELIN-1192] Block pyspark paragraph hang.

2016-08-15 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1193 \cc @bzz 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 project does not have this feature enabled and wishes so

[GitHub] zeppelin issue #1318: [ZEPPELIN-1280][Spark on Yarn] Documents for running z...

2016-08-16 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1318 I got a following error when I try to run zeppelin with spark2.0&hadoop2.7. ``` ERROR [2016-08-16 16:43:33,121] ({pool-1-thread-3} Utils.java[invokeMethod

[GitHub] zeppelin pull request #1335: [ZEPPELIN-1336] hadoop library for spark interp...

2016-08-16 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1335 [ZEPPELIN-1336] hadoop library for spark interpreter is not match. ### What is this PR for? This PR is for download hadoop libraries for yarn cluster. This issue comes from https

[GitHub] zeppelin issue #1318: [ZEPPELIN-1280][Spark on Yarn] Documents for running z...

2016-08-16 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1318 @felixcheung Yes 2.2, Maybe it's because my maven repo has different versions of hadoop libraries like following. ``` ~/zeppelin$ ls -al ~/.m2/repository/org/apache/hadoop/hadoop-c

[GitHub] zeppelin issue #1318: [ZEPPELIN-1280][Spark on Yarn] Documents for running z...

2016-08-16 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1318 I got success zeppelin job with `spark2.0&hadoop2.7` after apply https://github.com/apache/zeppelin/pull/1335. - hadoop libraries. ``` ~/zeppelin$ ls -al ./spark/target

[GitHub] zeppelin pull request #1193: [ZEPPELIN-1192] Block pyspark paragraph hang.

2016-08-16 Thread astroshim
GitHub user astroshim reopened a pull request: https://github.com/apache/zeppelin/pull/1193 [ZEPPELIN-1192] Block pyspark paragraph hang. ### What is this PR for? This PR block pyspark paragraph hang. ### What type of PR is it? Bug Fix ### What is

[GitHub] zeppelin pull request #1193: [ZEPPELIN-1192] Block pyspark paragraph hang.

2016-08-16 Thread astroshim
Github user astroshim closed the pull request at: https://github.com/apache/zeppelin/pull/1193 --- 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 #1193: [ZEPPELIN-1192] Block pyspark paragraph hang.

2016-08-16 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1193 The reason of CI fails is.. ``` Resolving deltas: 100% (13946/13946), completed with 385 local objects. From https://github.com/apache/zeppelin * branchrefs/pull/1193

[GitHub] zeppelin issue #1302: [ZEPPELIN-1306] Interpreter restarts on a note.

2016-08-16 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1302 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 project does not have this feature enabled and wishes so, or if the

[GitHub] zeppelin issue #1318: [ZEPPELIN-1280][Spark on Yarn] Documents for running z...

2016-08-16 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1318 @zjffdu I just tested and got success with removing the yarn profile. I will close the https://github.com/apache/zeppelin/pull/1335 then. Thank you. --- If your project is set up for

[GitHub] zeppelin issue #1335: [ZEPPELIN-1336] hadoop library for spark interpreter i...

2016-08-16 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1335 @jongyoul I will close this issue since removing yarn profile is working. please do it at the https://github.com/apache/zeppelin/pull/1301 as @zjffdu mentioned. --- If your project is set

[GitHub] zeppelin pull request #1335: [ZEPPELIN-1336] hadoop library for spark interp...

2016-08-16 Thread astroshim
Github user astroshim closed the pull request at: https://github.com/apache/zeppelin/pull/1335 --- 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 #1318: [ZEPPELIN-1280][Spark on Yarn] Documents for running z...

2016-08-17 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1318 Please merge this if there is no more discussion because I want to make document for https://issues.apache.org/jira/browse/ZEPPELIN-1279. --- If your project is set up for it, you can reply to

[GitHub] zeppelin issue #1318: [ZEPPELIN-1280][Spark on Yarn] Documents for running z...

2016-08-17 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1318 @zjffdu You're right, usually users don't make their production using docker. but We have been received many question about running Zeppelin on their production environments so I

[GitHub] zeppelin issue #1318: [ZEPPELIN-1280][Spark on Yarn] Documents for running z...

2016-08-17 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1318 I can update PR title but This PR is one of the https://issues.apache.org/jira/browse/ZEPPELIN-1198 and https://issues.apache.org/jira/browse/ZEPPELIN-1278 is already merged. and

[GitHub] zeppelin issue #1284: [ZEPPELIN-1291]Tagging meta value on interpreter compl...

2016-08-22 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1284 re-trigger CI --- 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 #1284: [ZEPPELIN-1291]Tagging meta value on interprete...

2016-08-22 Thread astroshim
GitHub user astroshim reopened a pull request: https://github.com/apache/zeppelin/pull/1284 [ZEPPELIN-1291]Tagging meta value on interpreter completion. ### What is this PR for? This PR is for showing the meta tag on auto completion. ### What type of PR is it

[GitHub] zeppelin pull request #1284: [ZEPPELIN-1291]Tagging meta value on interprete...

2016-08-22 Thread astroshim
Github user astroshim closed the pull request at: https://github.com/apache/zeppelin/pull/1284 --- 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 #1302: [ZEPPELIN-1306] Interpreter restarts on a note.

2016-08-22 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1302 Thank you very much about reviewing this PR. :smile: --- 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] zeppelin pull request #1358: [ZEPPELIN-1365] Error of Zeppelin Application i...

2016-08-24 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1358 [ZEPPELIN-1365] Error of Zeppelin Application in development mode. ### What is this PR for? This PR fixes the bug of running application in development mode. ### What type of

[GitHub] zeppelin issue #1302: [ZEPPELIN-1306] Interpreter restarts on a note.

2016-08-24 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1302 @Leemoonsoo Your suggestion about UI is definitely better! and also makes sense about restarting action on interpreter binding page. I will fix them. --- If your project is set up for

[GitHub] zeppelin pull request #1358: [ZEPPELIN-1365] Error of Zeppelin Application i...

2016-08-25 Thread astroshim
GitHub user astroshim reopened a pull request: https://github.com/apache/zeppelin/pull/1358 [ZEPPELIN-1365] Error of Zeppelin Application in development mode. ### What is this PR for? This PR fixes the bug of running application in development mode. ### What type

[GitHub] zeppelin issue #1358: [ZEPPELIN-1365] Error of Zeppelin Application in devel...

2016-08-25 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1358 re-trigger CI --- 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 #1358: [ZEPPELIN-1365] Error of Zeppelin Application i...

2016-08-25 Thread astroshim
Github user astroshim closed the pull request at: https://github.com/apache/zeppelin/pull/1358 --- 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 #1366: ZEPPELIN-1342. Adding dependencies via SPARK_SUBMIT_OP...

2016-08-26 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1366 LGTM. :+1: --- 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 #1302: [ZEPPELIN-1306] Interpreter restarts on a note.

2016-08-27 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1302 \cc @Leemoonsoo I fixed UI like following screenshot and process. please review. ![o](https://cloud.githubusercontent.com/assets/3348133/18028272/33d20810-6cb5-11e6-9026-b51ddf8eb972.gif

[GitHub] zeppelin issue #1302: [ZEPPELIN-1306] Interpreter restarts on a note.

2016-08-28 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1302 @AhyoungRyu I agree with you! I changed position and restart icon to button type, is it okay with button or just icon is better? ![button](https://cloud.githubusercontent.com

[GitHub] zeppelin issue #1302: [ZEPPELIN-1306] Interpreter restarts on a note.

2016-08-28 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1302 I just changed to 'icon'. Thank you. --- 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] zeppelin issue #1358: [ZEPPELIN-1365] Error of Zeppelin Application in devel...

2016-08-28 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1358 @AhyoungRyu Thank you for reviewing!! Did you clear your browser cache? --- 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] zeppelin issue #1358: [ZEPPELIN-1365] Error of Zeppelin Application in devel...

2016-08-30 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1358 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 project does not have this feature enabled and wishes so, or if the

[GitHub] zeppelin issue #1302: [ZEPPELIN-1306] Interpreter restarts on a note.

2016-08-30 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1302 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 project does not have this feature enabled and wishes so, or if the

[GitHub] zeppelin pull request #1389: [ZEPPELIN-1279] Spark on Mesos Docker.

2016-08-30 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1389 [ZEPPELIN-1279] Spark on Mesos Docker. ### What is this PR for? This PR is for the documentation of running zeppelin on production environments especially spark on mesos via Docker

[GitHub] zeppelin issue #1389: [ZEPPELIN-1279] Spark on Mesos Docker.

2016-08-30 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1389 @AhyoungRyu Thank you for quick review. I fixed what you pointed out. --- 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 issue #1385: [ZEPPELIN-1395] Local or Remote Interpreter by Configu...

2016-08-31 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1385 Please fix the CI-fails. ``` [INFO] There are 5 checkstyle errors. [ERROR] InterpreterFactory.java[302] (sizes) LineLength: Line is longer than 100 characters (found 106

[GitHub] zeppelin issue #1369: [ZEPPELIN-1376] Add proxy credentials for dependency r...

2016-08-31 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1369 About the second commits, Is it intended behavior that the error message bar is stacked whenever save the other interpreter setting? ![image](https://cloud.githubusercontent.com/assets

[GitHub] zeppelin issue #1369: [ZEPPELIN-1376] Add proxy credentials for dependency r...

2016-08-31 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1369 Wow very quick response!!: I think set `combineDuplications` true is better but I don't have much talent in UI.:) --- If your project is set up for it, you can reply to this emai

[GitHub] zeppelin issue #1302: [ZEPPELIN-1306] Interpreter restarts on a note.

2016-08-31 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1302 @AhyoungRyu I think `deactivated` button is better so I updated it. 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

[GitHub] zeppelin issue #1358: [ZEPPELIN-1365] Error of Zeppelin Application in devel...

2016-09-01 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1358 Thank you @corneadoug --- 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 #1369: [ZEPPELIN-1376] Add proxy credentials for dependency r...

2016-09-01 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1369 @doanduyhai Good. I am going to check first commit(proxy) soon. --- 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] zeppelin issue #1369: [ZEPPELIN-1376] Add proxy credentials for dependency r...

2016-09-01 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1369 @doanduyhai BTW how come only spark interpreter shows `error message bar`? --- 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] zeppelin issue #1389: [ZEPPELIN-1279] Spark on Mesos Docker.

2016-09-01 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1389 @AhyoungRyu Thank you!! :+1: --- 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 issue #1369: [ZEPPELIN-1376] Add proxy credentials for dependency r...

2016-09-01 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1369 Proxy repo tested and it works properly. :+1: --- 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 issue #1302: [ZEPPELIN-1306] Interpreter restarts on a note.

2016-09-02 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1302 @corneadoug Actually it's very short time to restart interpreter job because it's just kill interpreter process or thread. How about alerting message when request is success? -

[GitHub] zeppelin pull request #1396: [ZEPPELIN-1405] Remove propertyKeyUnusedConnect...

2016-09-02 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1396 [ZEPPELIN-1405] Remove propertyKeyUnusedConnectionListMap variable in JDBCInterpreter ### What is this PR for? This PR is for refactoring code for JDBCInterpreter. ### What

[GitHub] zeppelin issue #1396: [ZEPPELIN-1405] Remove propertyKeyUnusedConnectionList...

2016-09-02 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1396 \cc @jongyoul Please review this PR if i misunderstand. --- 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] zeppelin issue #1396: [ZEPPELIN-1405] Remove propertyKeyUnusedConnectionList...

2016-09-02 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1396 @jongyoul I will refactor the code to use database pool. --- 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] zeppelin issue #1396: [ZEPPELIN-1405] Remove propertyKeyUnusedConnectionList...

2016-09-02 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1396 I will do it in this PR and change the title this. :) --- 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] zeppelin issue #1369: [ZEPPELIN-1376] Add proxy credentials for dependency r...

2016-09-05 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1369 @doanduyhai Could you check the infinitely downloading library problem(like following screen shot)? Strangely this problem happens all interpreter except spark. Is this problem happen only

[GitHub] zeppelin issue #1396: [WIP] [ZEPPELIN-1405] ConnectionPool for JDBCInterpret...

2016-09-05 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1396 \cc @jongyoul I changed some codes to use `Connection Pool` that is `org.apache.commons.dbcp2`, please review. --- If your project is set up for it, you can reply to this email and have your

[GitHub] zeppelin issue #1302: [ZEPPELIN-1306] Interpreter restarts on a note.

2016-09-05 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1302 @corneadoug I fixed it. please review. Thank you! --- 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 #1396: [ZEPPELIN-1405] ConnectionPool for JDBCInterpreter.

2016-09-05 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1396 I tested `mysql`, `postgresql` and `hive`. --- 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 issue #1396: [ZEPPELIN-1405] ConnectionPool for JDBCInterpreter.

2016-09-06 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1396 Thanks @jongyoul and there was no problem running mysql. --- 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] zeppelin pull request #1411: change to create local interpreter when option ...

2016-09-07 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1411 change to create local interpreter when option is false. ### What is this PR for? This PR fixes [ZEPPELIN-1417](https://issues.apache.org/jira/browse/ZEPPELIN-1417 ). ### What

[GitHub] zeppelin issue #1411: [ZEPPELIN-1417] Bugfix of "Connect to existing process...

2016-09-07 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1411 @bzz Thank you for review and sorry for making you confused. I'll fix version to 0.7.0. --- If your project is set up for it, you can reply to this email and have your reply appe

[GitHub] zeppelin issue #1396: [ZEPPELIN-1405] ConnectionPool for JDBCInterpreter.

2016-09-07 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1396 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 project does not have this feature enabled and wishes so, or if the

[GitHub] zeppelin issue #1358: [ZEPPELIN-1365] Error of Zeppelin Application in devel...

2016-09-08 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1358 @mikhaildubkov `dev` doesn't exists on 0.6.x so you should use `master` branch for this. Thanks. --- If your project is set up for it, you can reply to this email and have your

[GitHub] zeppelin issue #1302: [ZEPPELIN-1306] Interpreter restarts on a note.

2016-09-08 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1302 @corneadoug I just re-based 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

[GitHub] zeppelin pull request #1396: [ZEPPELIN-1405] ConnectionPool for JDBCInterpre...

2016-09-09 Thread astroshim
Github user astroshim closed the pull request at: https://github.com/apache/zeppelin/pull/1396 --- 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 #1396: [ZEPPELIN-1405] ConnectionPool for JDBCInterpreter.

2016-09-09 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1396 re-trigger CI --- 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 #1396: [ZEPPELIN-1405] ConnectionPool for JDBCInterpre...

2016-09-09 Thread astroshim
GitHub user astroshim reopened a pull request: https://github.com/apache/zeppelin/pull/1396 [ZEPPELIN-1405] ConnectionPool for JDBCInterpreter. ### What is this PR for? This PR is for refactoring code for JDBCInterpreter. There is no putting 'Connectio

[GitHub] zeppelin issue #1358: [ZEPPELIN-1365] Error of Zeppelin Application in devel...

2016-09-11 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1358 Sorry maybe it's my mistake to set jira version. --- 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] zeppelin issue #1396: [ZEPPELIN-1405] ConnectionPool for JDBCInterpreter.

2016-09-12 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1396 re-trigger CI --- 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 #1396: [ZEPPELIN-1405] ConnectionPool for JDBCInterpre...

2016-09-12 Thread astroshim
Github user astroshim closed the pull request at: https://github.com/apache/zeppelin/pull/1396 --- 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 #1396: [ZEPPELIN-1405] ConnectionPool for JDBCInterpre...

2016-09-12 Thread astroshim
GitHub user astroshim reopened a pull request: https://github.com/apache/zeppelin/pull/1396 [ZEPPELIN-1405] ConnectionPool for JDBCInterpreter. ### What is this PR for? This PR is for refactoring code for JDBCInterpreter. There is no putting 'Connectio

[GitHub] zeppelin issue #1369: [ZEPPELIN-1376] Add proxy credentials for dependency r...

2016-09-16 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1369 @doanduyhai Sorry to keep you waiting. It works like a charm now but the error count number at the beginning of message appears a little strange like following screen shot. ![st](https

[GitHub] zeppelin issue #1369: [ZEPPELIN-1376] Add proxy credentials for dependency r...

2016-09-16 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1369 @doanduyhai I know that number means but as you can see the screenshot i attached, the number seems doesn't match with count. --- If your project is set up for it, you can reply to this

[GitHub] zeppelin issue #1302: [ZEPPELIN-1306] Interpreter restarts on a note.

2016-09-16 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1302 @Leemoonsoo I agree with your opinion. Let me fix them. --- 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] zeppelin pull request #1431: [ZEPPELIN-1446] Fix broken layout of `Create ne...

2016-09-16 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1431 [ZEPPELIN-1446] Fix broken layout of `Create new interpreter` UI. ### What is this PR for? This PR fixes broken layout of `Create new interpreter` UI. ### What type of PR is

[GitHub] zeppelin issue #1369: [ZEPPELIN-1376] Add proxy credentials for dependency r...

2016-09-18 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1369 @doanduyhai Could you take a look at the following screenshot? After the `alluxio` and `angular` dependency error occurs, I just saved `alluxio` interpreter setting several times but the

[GitHub] zeppelin pull request #1433: [ZEPPELIN-1451] Bug fix of Embedding %html with...

2016-09-18 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1433 [ZEPPELIN-1451] Bug fix of Embedding %html within %table. ### What is this PR for? This PR fixes the bug of Embedding %html within %table. It doesn't print properly when the `%htm

[GitHub] zeppelin issue #1431: [ZEPPELIN-1446] Fix broken layout of `Create new inter...

2016-09-18 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1431 @AhyoungRyu Thank you for your kind review. I just fix the space. --- 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 issue #1431: [ZEPPELIN-1446] Fix broken layout of `Create new inter...

2016-09-19 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1431 Thank you @AhyoungRyu and @corneadoug . I just fixed space problems. --- 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 issue #1433: [ZEPPELIN-1451] Bug fix of Embedding %html within %tab...

2016-09-21 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1433 @corneadoug Thank you for your review. I just re-based. --- 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] zeppelin issue #1433: [ZEPPELIN-1451] Bug fix of Embedding %html within %tab...

2016-09-21 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1433 @Leemoonsoo Thank you for pointing out the problem. Let me check. --- 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] zeppelin issue #1431: [ZEPPELIN-1446] Fix broken layout of `Create new inter...

2016-09-21 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1431 I rebased but build failed. --- 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 #1431: [ZEPPELIN-1446] Fix broken layout of `Create ne...

2016-09-22 Thread astroshim
Github user astroshim closed the pull request at: https://github.com/apache/zeppelin/pull/1431 --- 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 #1431: [ZEPPELIN-1446] Fix broken layout of `Create ne...

2016-09-22 Thread astroshim
GitHub user astroshim reopened a pull request: https://github.com/apache/zeppelin/pull/1431 [ZEPPELIN-1446] Fix broken layout of `Create new interpreter` UI. ### What is this PR for? This PR fixes broken layout of `Create new interpreter` UI. ### What type of PR is

[GitHub] zeppelin issue #1431: [ZEPPELIN-1446] Fix broken layout of `Create new inter...

2016-09-22 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1431 re-build CI --- 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 #1431: [ZEPPELIN-1446] Fix broken layout of `Create new inter...

2016-09-22 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1431 re-build CI --- 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 pull request #1431: [ZEPPELIN-1446] Fix broken layout of `Create ne...

2016-09-22 Thread astroshim
GitHub user astroshim reopened a pull request: https://github.com/apache/zeppelin/pull/1431 [ZEPPELIN-1446] Fix broken layout of `Create new interpreter` UI. ### What is this PR for? This PR fixes broken layout of `Create new interpreter` UI. ### What type of PR is

[GitHub] zeppelin pull request #1431: [ZEPPELIN-1446] Fix broken layout of `Create ne...

2016-09-22 Thread astroshim
Github user astroshim closed the pull request at: https://github.com/apache/zeppelin/pull/1431 --- 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 #1451: [ZEPPELIN-1279] Zeppelin with CDH5.x docker doc...

2016-09-22 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1451 [ZEPPELIN-1279] Zeppelin with CDH5.x docker document. ### What is this PR for? This PR is for the documentation of running zeppelin with CDH docker environment. and This PR is the part

[GitHub] zeppelin pull request #1396: [ZEPPELIN-1405] ConnectionPool for JDBCInterpre...

2016-09-22 Thread astroshim
GitHub user astroshim reopened a pull request: https://github.com/apache/zeppelin/pull/1396 [ZEPPELIN-1405] ConnectionPool for JDBCInterpreter. ### What is this PR for? This PR is for refactoring code for JDBCInterpreter. There is no putting 'Connectio

[GitHub] zeppelin pull request #1396: [ZEPPELIN-1405] ConnectionPool for JDBCInterpre...

2016-09-22 Thread astroshim
Github user astroshim closed the pull request at: https://github.com/apache/zeppelin/pull/1396 --- 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

<    1   2   3   4   5   6   >