[GitHub] incubator-zeppelin pull request: untracked files configuration.

2015-08-23 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/incubator-zeppelin/pull/248 untracked files configuration. To fix https://issues.apache.org/jira/browse/ZEPPELIN-258. You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] incubator-zeppelin pull request: untracked files configuration.

2015-08-24 Thread astroshim
Github user astroshim closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/248 --- 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: untracked files configuration.

2015-08-31 Thread astroshim
GitHub user astroshim reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/248 untracked files configuration. To fix https://issues.apache.org/jira/browse/ZEPPELIN-258. You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] incubator-zeppelin pull request: untracked files configuration.

2015-08-31 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/248#issuecomment-136536350 Not merged yet.:) --- 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: ZEPPELIN-419 Change loglevel to d...

2015-11-13 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/428#issuecomment-156469757 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

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-37] Upload to sonatype ...

2015-11-20 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/incubator-zeppelin/pull/453 [ZEPPELIN-37] Upload to sonatype maven repository the spark interpreter module. Since maven-deploy-plugin is set as skip in the spark pom.xml, it'll be removed. related

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-37] Upload to sonatype ...

2015-11-20 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/453#issuecomment-158581828 My build was passed(below log is my build result), why travis has failed? ``` [INFO] Reactor Summary: [INFO] [INFO] Zeppelin

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-37] Upload to sonatype ...

2015-11-21 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/453#issuecomment-158614518 @Leemoonsoo I got 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 your

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-37] Upload to sonatype ...

2015-11-21 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/453#issuecomment-158629282 Travis build failed at the "mvn verify -Pusing-packaged-distr -Phadoop-2.3 -Ppyspark -B" command but my update is not relation about it i think

[GitHub] incubator-zeppelin pull request: Improve storing notebook.

2015-11-21 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/incubator-zeppelin/pull/457 Improve storing notebook. This PR relates to: https://issues.apache.org/jira/browse/ZEPPELIN-454 Storing the notebook process changed. (store".note.json" temporary file

[GitHub] incubator-zeppelin pull request: Add job operation API

2015-11-23 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/incubator-zeppelin/pull/465 Add job operation API This PR is related to https://issues.apache.org/jira/browse/ZEPPELIN-137. I added some notebook job operations. Here is the examples. * Get

[GitHub] incubator-zeppelin pull request: Add job operation API

2015-11-23 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/465#issuecomment-159170573 @bzz Thank you for your comment. But I don't understand what you mean the java style, could you tell me more detail? --- If your project is s

[GitHub] incubator-zeppelin pull request: Add job operation API

2015-11-23 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/465#issuecomment-159171649 @springml Thank you for your comment. What is the runtime parameter? Is runtime you mean is the time will be executed? --- If your project is set up

[GitHub] incubator-zeppelin pull request: Add job operation API

2015-11-24 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/465#issuecomment-159196978 @springml I am going to add more APIs related to scehdule job. That could be solution. --- If your project is set up for it, you can reply to this email

[GitHub] incubator-zeppelin pull request: Add job operation API

2015-11-24 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/465#issuecomment-159463022 @bzz Your comment is very helpful. Thank you very much!! --- 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: Add job operation API

2015-11-25 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/465#issuecomment-159808250 I added cron job api's - Add cron job ``` ]# curl -XPOST http://121.156.59.2:8080/api/notebook/cron/2B79J5UQB -d &#

[GitHub] incubator-zeppelin pull request: synchronization saving notebook.

2015-11-30 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/incubator-zeppelin/pull/489 synchronization saving notebook. The issue related to https://issues.apache.org/jira/browse/ZEPPELIN-473. Saving notebook operation has called save method in the VFSNotebookRepo

[GitHub] incubator-zeppelin pull request: synchronization saving notebook.

2015-11-30 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/489#issuecomment-160642541 @bzz Thanks for comment. I thought delete mothod can be needed the synchronization but because of delete method just occur exception when it

[GitHub] incubator-zeppelin pull request: synchronization saving notebook.

2015-11-30 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/489#issuecomment-160859692 I implemented some code for testing. As you can see, In the while() if saving note without synchronization, note.json is crashed. (disappeared

[GitHub] incubator-zeppelin pull request: synchronization saving notebook.

2015-11-30 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/489#issuecomment-160884961 @bzz Thanks for replying. The code is the VFSNotebookRepoTest.java but i didn't push yet. I'm wondering is that can't show the

[GitHub] incubator-zeppelin pull request: synchronization saving notebook.

2015-12-01 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/489#issuecomment-160916885 @bzz Thank you very much!! It seems like CI fails is not related to testcase. What do you think? --- If your project is set up for it, you can reply

[GitHub] incubator-zeppelin pull request: synchronization saving notebook.

2015-12-02 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/489#issuecomment-161515202 @bzz Okay. I will. Thank you very much for your help~:) --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] incubator-zeppelin pull request: Add job operation API

2015-12-08 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/465#issuecomment-162895268 @Leemoonsoo @bzz Thank you for your review! I totally agree with your idea about consistent statuses. I'll fix them. --- If your project is set u

[GitHub] incubator-zeppelin pull request: Add job operation API

2015-12-09 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/465#issuecomment-163287793 @bzz I updated REST API docs, could you review? Thanks. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] incubator-zeppelin pull request: ZEPPELIN-518 ] always top fixed c...

2015-12-18 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/554#issuecomment-165888962 This function is very useful for users! And the code what you added is the brilliant trick i think. :) I tested this function at chrome/firefox

[GitHub] incubator-zeppelin pull request: [Improve] Add interpreter restart...

2015-12-18 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/incubator-zeppelin/pull/555 [Improve] Add interpreter restart REST API doc ### What is this PR for? Add restarting interpreter REST API information to the document. There is no information in the

[GitHub] incubator-zeppelin pull request: [Improve] Add interpreter restart...

2015-12-18 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/555#issuecomment-165946523 @bzz 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

[GitHub] incubator-zeppelin pull request: ZEPPELIN-523 ] table scrolling bu...

2015-12-19 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/556#issuecomment-165959908 Cool fix! I tested on chrome/firefox/explore10 and working well. Thanks. --- If your project is set up for it, you can reply to this email and

[GitHub] incubator-zeppelin pull request: [Improve] Add interpreter restart...

2015-12-20 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/555#issuecomment-166166917 Build has passed. please merge 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

[GitHub] incubator-zeppelin pull request: Release resource after cron job.

2015-12-20 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/incubator-zeppelin/pull/557 Release resource after cron job. ### What is this PR for? Release resource after cron schedule job. ### What type of PR is it? Improvement ### Todos

[GitHub] incubator-zeppelin pull request: Release resource after cron job.

2015-12-20 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/557#issuecomment-166184648 Thank @Leemoonsoo your feedback. I'm going to implement the unittest. --- If your project is set up for it, you can reply to this email and have

[GitHub] incubator-zeppelin pull request: Release resource after cron job.

2015-12-21 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/557#issuecomment-166234115 @bzz Thanks your feedback. Yes, "release resource" means restarting all binded interpreters on the note. I was thinking rest

[GitHub] incubator-zeppelin pull request: Release resource after cron job.

2015-12-23 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/557#issuecomment-166837719 Dear @bzz I totally agree with you. I'll update the text and testcase. Thanks! --- If your project is set up for it, you can reply to this

[GitHub] incubator-zeppelin pull request: Auto-restart interpreters on cron...

2015-12-25 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/557#issuecomment-167243619 Hello. @felixcheung You're right.. The interpreter process is shared now but I think interpreter process will be separated as user option.

[GitHub] incubator-zeppelin pull request: Fix restapi testcase

2015-12-27 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/incubator-zeppelin/pull/572 Fix restapi testcase ### What is this PR for? To fix REST API testcase. ### What type of PR is it? Bug Fix ### Is there a relevant Jira issue? https

[GitHub] incubator-zeppelin pull request: Fix restapi testcase

2015-12-27 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/572#issuecomment-167461109 @HeartSaVioR Thank you for your reply. --- 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: Fix restapi testcase

2015-12-27 Thread astroshim
Github user astroshim closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/572 --- 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: Fix restapi testcase

2015-12-27 Thread astroshim
GitHub user astroshim reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/572 Fix restapi testcase ### What is this PR for? To fix REST API testcase. ### What type of PR is it? Bug Fix ### Is there a relevant Jira issue? https

[GitHub] incubator-zeppelin pull request: Auto-restart interpreters on cron...

2015-12-27 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/557#issuecomment-167461320 @Leemoonsoo Thank you for making clear. --- 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: Fix restapi testcase

2015-12-29 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/572#issuecomment-167776170 The build error is not related to this issue. But it occurred again! ``` ... 15/12/28 04:23:52 WARN TaskSchedulerImpl: Initial job has not

[GitHub] incubator-zeppelin pull request: [WIP][Zeppelin-630] Introduce new...

2016-01-25 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/673#issuecomment-174815976 This is really nice feature and a lot of helpful for users. I'll review this. :) Thank you. --- If your project is set up for it, you can rep

[GitHub] incubator-zeppelin pull request: [WIP][Zeppelin-630] Introduce new...

2016-01-26 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/673#issuecomment-174947906 There are some bugs. 1. It seems the URL string does not apply. - as you can see, URL value to be blanked. ![1](https

[GitHub] incubator-zeppelin pull request: [WIP][Zeppelin-630] Introduce new...

2016-01-26 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/673#issuecomment-175354290 Tested and working well. :+1: Cool! --- 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: Fix nginx config to use single po...

2016-04-04 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/incubator-zeppelin/pull/814 Fix nginx config to use single port for websocket and REST. ### What is this PR for? Zeppelin uses single port number for REST and websocket since 0.5.5 so It should be fixed

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-799] Add python edit mo...

2016-04-08 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/incubator-zeppelin/pull/825 [ZEPPELIN-799] Add python edit mode. ### What is this PR for? support python edit mode in paragraph. ### What type of PR is it? Improvement ### Todos

[GitHub] incubator-zeppelin pull request: ZEPPELIN-830 Improve table displa...

2016-04-26 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/858#issuecomment-214765937 I passed the build with no problems and functionality is working very well. but @r-kamath you should make the green CI. Thanks. --- If your

[GitHub] incubator-zeppelin pull request: ZeppelinHub notebook storage/conn...

2016-05-20 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/880#issuecomment-220565542 It works like a charm. --- 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-164 Fixing typo on the z...

2015-08-13 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/incubator-zeppelin/pull/209 ZEPPELIN-164 Fixing typo on the zeppelin website. https://issues.apache.org/jira/browse/ZEPPELIN-164 Here is the fixed screen. (locally tested.) - before ![image