[GitHub] incubator-zeppelin pull request: [ZEPPELIN-835]Don't persist authe...

2016-05-19 Thread mwkang
GitHub user mwkang opened a pull request: https://github.com/apache/incubator-zeppelin/pull/904 [ZEPPELIN-835]Don't persist authentication info into note.json ### What is this PR for? Don't persist authentication info into note.json ### What type of PR is i

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-707]Automatically adds ...

2016-05-17 Thread mwkang
Github user mwkang commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/806#issuecomment-219896471 @corneadoug @Leemoonsoo Yeah! I passed all test. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-707]Automatically adds ...

2016-05-08 Thread mwkang
Github user mwkang commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/806#issuecomment-217761459 I'm so sorry. CI was failed. I'll try to fix it. --- 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-707]Automatically adds ...

2016-05-01 Thread mwkang
Github user mwkang commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/806#issuecomment-216021157 @corneadoug I fixed that problems. Could you mind if I request review to you? --- If your project is set up for it, you can reply to this email and

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-707]Automatically adds ...

2016-04-29 Thread mwkang
Github user mwkang commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/806#issuecomment-215705028 @corneadoug Yes, Sorry for delay. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-707]Automatically adds ...

2016-04-10 Thread mwkang
Github user mwkang commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/806#issuecomment-208182170 @corneadoug Could you review 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: [ZEPPELIN-707]Automatically adds ...

2016-04-04 Thread mwkang
Github user mwkang commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/806#issuecomment-205164365 I change my code it is use only backend. I force push my remote branch. So remove previous commit. --- If your project is set up for it, you can reply to

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-707]Automatically adds ...

2016-04-03 Thread mwkang
Github user mwkang commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/806#issuecomment-205158691 @corneadoug I have a question. If User doesn't run paragraph, Always add `%..`. Because I couldn't know what is recent usage of the interpreter(by

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-707]Automatically adds ...

2016-04-03 Thread mwkang
Github user mwkang commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/806#issuecomment-205102549 @bzz Thank you for your review. @corneadoug Okay, I will change the code does not use a front-end code. --- If your project is set up for it, you can

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-707]Automatically adds ...

2016-04-02 Thread mwkang
Github user mwkang commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/806#issuecomment-204681172 Is there anything wrong? --- 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-707]Automatically adds ...

2016-04-02 Thread mwkang
Github user mwkang commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/806#issuecomment-204680411 @felixcheung Yes, it works. If user doesn't any %interpreter in previous paragraphs, `%..`(default interpreter) will be added. If use

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-707]Automatically adds ...

2016-03-31 Thread mwkang
Github user mwkang commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/806#issuecomment-203810062 The purpose of this front-end code is to track the recent usage of the interpteter(by the users). Okay. include the `%...` in the empty note. --- If

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-707]Automatically adds ...

2016-03-30 Thread mwkang
Github user mwkang commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/806#issuecomment-203746988 @Leemoonsoo I change some code. Could you mind if I request to review this? --- If your project is set up for it, you can reply to this email and have

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-707]Automatically adds ...

2016-03-30 Thread mwkang
Github user mwkang commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/806#issuecomment-203707670 I agree with you. This commit is too hasty. I'm so sorry. I'm looking in that. --- If your project is set up for it, you can reply to this emai

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-707]Automatically adds ...

2016-03-29 Thread mwkang
GitHub user mwkang opened a pull request: https://github.com/apache/incubator-zeppelin/pull/806 [ZEPPELIN-707]Automatically adds %.* of previous paragraph ### What is this PR for? Automatically adds %pyspark in the code area after paragraph created, if previous paragrpah is

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-754]Cannot use S3 seoul...

2016-03-23 Thread mwkang
Github user mwkang commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/791#issuecomment-200649714 Oh I'm so sorry. I updated the file. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-715]Provide version inf...

2016-03-23 Thread mwkang
Github user mwkang commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/792#issuecomment-200638273 Okay. I changed the command line arguments. --- 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: [ZEPPELIN-715]Provide version inf...

2016-03-23 Thread mwkang
GitHub user mwkang opened a pull request: https://github.com/apache/incubator-zeppelin/pull/792 [ZEPPELIN-715]Provide version information ### What is this PR for? Provides version information from it's REST api, GUI and command line ### What type of PR

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-754]Cannot use S3 seoul...

2016-03-22 Thread mwkang
GitHub user mwkang opened a pull request: https://github.com/apache/incubator-zeppelin/pull/791 [ZEPPELIN-754]Cannot use S3 seoul region (ap-northeast-2) ### What is this PR for? Update AWS SDK version. ### What type of PR is it? Bug Fix ### What is

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-751]Log an error and co...

2016-03-22 Thread mwkang
Github user mwkang commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/787#issuecomment-200116775 Okay. I remove last commit. --- 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-751]Log an error and co...

2016-03-21 Thread mwkang
Github user mwkang commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/787#issuecomment-199574692 Oh.. It is my fault. This request includes also [ZEPPELIN-754]Support seoul region. Because I push '0d98c98' commit before Close this reqeu

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-751]Log an error and co...

2016-03-21 Thread mwkang
GitHub user mwkang opened a pull request: https://github.com/apache/incubator-zeppelin/pull/787 [ZEPPELIN-751]Log an error and continue reading the remaining notebooks ### What is this PR for? Log an Amazon Service Exception. Continue reading the remaining notebooks into

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-498]Manual upgrade proc...

2016-02-27 Thread mwkang
GitHub user mwkang opened a pull request: https://github.com/apache/incubator-zeppelin/pull/753 [ZEPPELIN-498]Manual upgrade procedure for Zeppelin ### What is this PR for? Describe Manual upgrade procedure for Zeppelin. ### What type of PR is it? Documentation

[GitHub] incubator-zeppelin pull request: ZEPPELIN-358: Jetty server is HTT...

2015-12-01 Thread mwkang
Github user mwkang commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/498#issuecomment-161161076 I update the documents. --- 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-358: Jetty server is HTT...

2015-12-01 Thread mwkang
Github user mwkang commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/498#issuecomment-161157672 Thanks for your review. I update the documents. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] incubator-zeppelin pull request: ZEPPELIN-358: Jetty server is HTT...

2015-12-01 Thread mwkang
GitHub user mwkang opened a pull request: https://github.com/apache/incubator-zeppelin/pull/498 ZEPPELIN-358: Jetty server is HTTP ERROR 404 Define local jetty temp directory. `varName: zeppelin.war.tempdir` `default varValue: webapps` Zeppelin create temp directory in