[GitHub] incubator-zeppelin pull request #964: filtering subdirectory names.

2016-06-05 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/incubator-zeppelin/pull/964 filtering subdirectory names. ### What is this PR for? It needs to filtering subdirectory names in navbar. ### What type of PR is it? Bug Fix

[GitHub] incubator-zeppelin pull request #979: Alert message when trying to delete la...

2016-06-09 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/incubator-zeppelin/pull/979 Alert message when trying to delete last paragraph. ### What is this PR for? This PR is for giving proper information to users and blocking unnecessary communication with zeppelin

[GitHub] incubator-zeppelin issue #919: support changing tabsize of paragraph

2016-06-09 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/incubator-zeppelin/pull/919 @corneadoug Thank you for your concern! I'm thinking to find right way how to handle this issue. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] zeppelin issue #979: Alert message when trying to delete last paragraph.

2016-06-13 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/979 @corneadoug Thank you for your kind explanation. I know the reason about the error but I couldn't reproduce the error even if i build like ```mvn clean package```. Did you build also

[GitHub] zeppelin issue #979: Alert message when trying to delete last paragraph.

2016-06-13 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/979 @corneadoug Okay. 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 this feature enabled

[GitHub] zeppelin issue #979: Alert message when trying to delete last paragraph.

2016-06-12 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/979 @minahlee Could you tell me how can i get your test failure? --- 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] zeppelin issue #964: filtering subdirectory names.

2016-06-10 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/964 @corneadoug I refactored the codes that you pointed out. please review. --- 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 #979: Alert message when trying to delete last paragraph.

2016-06-10 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/979 @corneadoug @minahlee I fixed the code to hide "Remove" menu on the last paragraph menu. please review. --- If your project is set up for it, you can reply to this email and have

[GitHub] zeppelin issue #964: filtering subdirectory names.

2016-06-10 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/964 @corneadoug Thank you for your review. I'll refactor the code as you mentioned. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] incubator-zeppelin issue #959: [ZEPPELIN-951] Update cron expression link an...

2016-06-03 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/incubator-zeppelin/pull/959 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 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 issue #1011: Update licenses

2016-06-14 Thread astroshim
Github user astroshim 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 issue #976: ZEPPLIN-976 ] HotFix - zeppelin server does not restart...

2016-06-14 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/976 LGTM. @minahlee The selenium test of travis has a problem so @corneadoug is fixing now. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] zeppelin issue #979: Alert message when trying to delete last paragraph.

2016-06-14 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/979 @corneadoug I really appreciate for your effort. Thank you. I reset my commits. --- 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 #965: [ZEPPELIN-957] Normalize note name.

2016-06-05 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/incubator-zeppelin/pull/965 [ZEPPELIN-957] Normalize note name. ### What is this PR for? It's better if notebook name can be normalized. ### What type of PR is it? Improvement

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

2016-06-08 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/incubator-zeppelin/pull/972 @cloverhearts Thank you for your explanation. You mean that the "LastResponseUnixTime" is for getting notebook information of lastest updated. --- If your project

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

2016-06-08 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/incubator-zeppelin/pull/972 It's working nicely. but Is the correct behavior that the lastResponseUnixTime is increased whenever call the curl? Then i can't understand what's the purpose of step 3. Could

[GitHub] incubator-zeppelin pull request: Hotfix - does not working noteboo...

2016-05-25 Thread astroshim
GitHub user astroshim reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/912 Hotfix - does not working notebook filter on navibar. ### What is this PR for? It doesn't working the notebook filter on navibar. ### What type of PR is it? Bug

[GitHub] incubator-zeppelin pull request: Hotfix - does not working noteboo...

2016-05-25 Thread astroshim
Github user astroshim closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/912 --- 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: ZEPPELIN-885 Add option to send e...

2016-05-25 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/918#issuecomment-221772165 @cloverhearts maybe you need to set up the smtp informations. you can set the variables in the "conf/zeppelin-site.xml". --- If your project

[GitHub] incubator-zeppelin pull request: support changing tabsize of parag...

2016-05-27 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/919#issuecomment-222195524 @Leemoonsoo It really makes sense and I thought that too. The reason why I thought about configuring tab size in notebook is better was just because

[GitHub] incubator-zeppelin pull request: Hotfix - does not working noteboo...

2016-05-26 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/912#issuecomment-221791290 @bzz I made issue(https://issues.apache.org/jira/browse/ZEPPELIN-886). --- If your project is set up for it, you can reply to this email and have your

[GitHub] incubator-zeppelin pull request: Hotfix - does not working noteboo...

2016-05-26 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/912#issuecomment-222044643 @minahlee It's my mistake. I reverted. --- 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: support changing tabsize of parag...

2016-05-26 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/919#issuecomment-222048031 @Leemoonsoo Good point. I agree with you that changing tab size in the interpreter or notebook is better than paragraph. so I think Configuring tab size

[GitHub] incubator-zeppelin issue #938: ZEPPELIN-930 ] fix width and not close li tag...

2016-06-01 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/incubator-zeppelin/pull/938 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 pull request: Zeppelin-531 ] Job management

2016-05-29 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/921#issuecomment-222408542 @cloverhearts Really nice job. but I got an follwing error when i click the paragraph in job ![image](https://cloud.githubusercontent.com/assets

[GitHub] incubator-zeppelin pull request: Hotfix - initial notebook name do...

2016-05-29 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/incubator-zeppelin/pull/929 Hotfix - initial notebook name does not increase. ### What is this PR for? To increase initial notebook name. ### What type of PR is it? Bug Fix

[GitHub] zeppelin issue #1058: BugFix of Automatically adds %.* of previous paragraph...

2016-06-22 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1058 The selenium test has passed!! --- 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 #1064: BugFix-blocking of blank values insertion on th...

2016-06-21 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1064 BugFix-blocking of blank values insertion on the Credential page. ### What is this PR for? This PR blocks the blank values insertion on the Credential page and changes the success

[GitHub] zeppelin issue #1064: BugFix-blocking of blank values insertion on the Crede...

2016-06-22 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1064 @bzz I fixed as your comments. please review. --- 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 #1058: BugFix of Automatically adds %.* of previous paragraph...

2016-06-22 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1058 @Leemoonsoo I rebased. --- 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 #1036: [MINOR] Add .Rhistory in .gitignore

2016-06-17 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1036 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 issue #919: support changing tabsize of paragraph

2016-06-21 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/919 Thank you for your detailed explanation @Leemoonsoo and I agree with moving interpreter-setting.json. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] zeppelin issue #1037: fix typo the description of interpreter menu.

2016-06-19 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1037 @minahlee Thanks for pointing out. I just fixed. --- 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 #1048: [ZEPPELIN-1033] HotFixing of paragraph deletion...

2016-06-20 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1048 [ZEPPELIN-1033] HotFixing of paragraph deletion and re-ordering broken ### What is this PR for? This PR is for fixing [this issue](https://issues.apache.org/jira/browse/ZEPPELIN-1033

[GitHub] zeppelin issue #1058: BugFix of Automatically adds %.* of previous paragraph...

2016-06-22 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1058 @Leemoonsoo Sorry for late response. I fixed. --- 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 #1048: [ZEPPELIN-1033] HotFixing of paragraph deletion and re...

2016-06-20 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1048 @randerzander Thank you for your review. 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 does

[GitHub] zeppelin issue #1030: [ZEPPELIN-1023] Add more credential apis.

2016-06-20 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1030 Thank you for your review @bzz and @AhyoungRyu and @cloverhearts ! And It really makes sense @bzz comments. Let me fix them. --- If your project is set up for it, you can reply

[GitHub] zeppelin issue #919: support changing tabsize of paragraph

2016-06-20 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/919 @bzz Thank you for your detail review! I was worring about changing tab size per interpreter as @Leemoonsoo suggests is more hard to config than per notebook or per paragraph. How about

[GitHub] zeppelin issue #1048: [ZEPPELIN-1033] HotFixing of paragraph deletion and re...

2016-06-20 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1048 nice @randerzander thanks for addressing this issue and review. --- 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 pull request #1058: BugFix of Automatically adds %.* of previous pa...

2016-06-21 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1058 BugFix of Automatically adds %.* of previous paragraph's typing. ### What is this PR for? This PR is to fix bug from https://github.com/apache/zeppelin/pull/806. The default interpreter

[GitHub] zeppelin pull request #1030: [ZEPPELIN-1023] Add more credential apis.

2016-06-16 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1030 [ZEPPELIN-1023] Add more credential apis. ### What is this PR for? This PR is for supporting various Credential APIs for users. ### What type of PR is it? Improvement

[GitHub] zeppelin issue #1079: Prevent menubar hide notebook title and control button...

2016-06-24 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1079 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 issue #1025: ZEPPELIN-964 ] Jobmanagement - (2) basic front-end

2016-06-17 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1025 @cloverhearts Thanks for the quick response. I understand. :+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

[GitHub] zeppelin pull request #1037: fix typo the description of interpreter menu.

2016-06-17 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1037 fix typo the description of interpreter menu. ### What is this PR for? This PR is for fixing typo of the interpreter menu description. ### What type of PR is it? Bug Fix

[GitHub] zeppelin issue #1030: [ZEPPELIN-1023] Add more credential apis.

2016-06-27 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1030 @bzz I fixed code and the build problem that you mentioned is gone. but another following build fail occurred. I think following problem seems not related this PR, what do you think

[GitHub] zeppelin pull request #1097: init newRepoSetting values

2016-06-27 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1097 init newRepoSetting values ### What is this PR for? This PR fixes initialize value bug of ```Add New Repository```. ### What type of PR is it? Bug Fix

[GitHub] zeppelin issue #1018: Blocking the zeppelin-web build error.

2016-06-15 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1018 It's not actual error just error messsage. --- 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 #1104: [ZEPPELIN-1078] Does not show the text result in the p...

2016-06-29 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1104 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 #1104: [ZEPPELIN-1078] Does not show the text result i...

2016-06-29 Thread astroshim
GitHub user astroshim reopened a pull request: https://github.com/apache/zeppelin/pull/1104 [ZEPPELIN-1078] Does not show the text result in the paragraph sometimes. ### What is this PR for? This PR fixes the problem that does not show the text result in the paragraph sometimes

[GitHub] zeppelin pull request #1104: [ZEPPELIN-1078] Does not show the text result i...

2016-06-29 Thread astroshim
Github user astroshim closed the pull request at: https://github.com/apache/zeppelin/pull/1104 --- 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 pull request: ZEPPELIN-598 ] Dynamic loading fo...

2016-05-23 Thread astroshim
Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/908#issuecomment-221145936 @cloverhearts I tested and really nice working. Thanks. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] zeppelin issue #919: support changing tabsize of paragraph

2016-07-26 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/919 https://github.com/apache/zeppelin/pull/1218 is better way. --- 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] zeppelin pull request #919: support changing tabsize of paragraph

2016-07-26 Thread astroshim
Github user astroshim closed the pull request at: https://github.com/apache/zeppelin/pull/919 --- 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 #1110: [ZEPPELIN-1091] Disable ace editor's showSettingsMenu ...

2016-07-12 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1110 Is this PR going to be merged? --- 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 #1185: Redirect home when requested a wrong notename path.

2016-07-14 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1185 @AhyoungRyu Thank you for your opinion. Your idea is great but now is redirected all to home when requested wrong path. (you can see the related code [here](https://github.com/apache

[GitHub] zeppelin pull request #1227: [ZEPPELIN-1198][Spark Standalone] Documents for...

2016-07-25 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1227 [ZEPPELIN-1198][Spark Standalone] Documents for running zeppelin on production environments. ### What is this PR for? This PR is for documentation for running zeppelin on production

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

2016-07-25 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1223 @prabhjyotsingh Thank you for share your great idea. This PR is for interpreter authorization but your idea seems include user management. I think it should be separated and I agree

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

2016-07-23 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1223 @Leemoonsoo Thank you for your review and I think your opinion makes sense. I'll change the UI. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] zeppelin issue #1117: [ZEPPELIN-1086] Auto Completion for Interpreter.

2016-07-21 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1117 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 pull request #1229: [BugFix] Show checkbox for "Connect to existing...

2016-08-01 Thread astroshim
Github user astroshim closed the pull request at: https://github.com/apache/zeppelin/pull/1229 --- 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 #1229: [BugFix] Show checkbox for "Connect to existing proces...

2016-08-01 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1229 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 #1229: [BugFix] Show checkbox for "Connect to existing...

2016-08-01 Thread astroshim
GitHub user astroshim reopened a pull request: https://github.com/apache/zeppelin/pull/1229 [BugFix] Show checkbox for "Connect to existing process" on interpreter menu. ### What is this PR for? This PR fixes a bug of showing checkbox for "Connect to e

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

2016-08-01 Thread astroshim
Github user astroshim closed the pull request at: https://github.com/apache/zeppelin/pull/1229 --- 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 #1229: [BugFix] Show checkbox for "Connect to existing proces...

2016-08-01 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1229 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 #1229: [BugFix] Show checkbox for "Connect to existing...

2016-08-01 Thread astroshim
GitHub user astroshim reopened a pull request: https://github.com/apache/zeppelin/pull/1229 [BugFix] Show checkbox for "Connect to existing process" on interpreter menu. ### What is this PR for? This PR fixes a bug of showing checkbox for "Connect to e

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

2016-07-26 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1229 [BugFix] Show checkbox for "Connect to existing process" on interpreter menu. ### What is this PR for? This PR fixes a bug of showing checkbox for "Connect to e

[GitHub] zeppelin issue #1248: [ZEPPELIN-1069]Ignore implicit interpreter when user e...

2016-08-02 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1248 tested. 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 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-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 se

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

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

2016-08-03 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1257 @AhyoungRyu I fixed you pointed out and I'll create the document for this separated PR. Thanks. --- If your project is set up for it, you can reply to this email and have your reply

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

2016-08-03 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1257 @AhyoungRyu Let me check. 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 issue #1257: [Zeppelin-945] Interpreter authorization

2016-08-04 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1257 @prabhjyotsingh Great. Thank you for giving me the guidance. --- 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] zeppelin issue #1257: [Zeppelin-945] Interpreter authorization

2016-08-04 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1257 Thank you for detail review @bzz and @felizbear I'll fix the codes. --- 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 #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

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

[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 #1171: [ZEPPELIN-1150] Table contents disappear 2nd run onwar...

2016-07-12 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1171 Build error has gone and working good! --- 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 #1176: [Zeppelin 1165 : improve] Paragraph dynamic relationsh...

2016-07-13 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1176 Great feature! Can you add documents for this or please explain more detail to review? --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] zeppelin issue #1117: [ZEPPELIN-1086] Auto Completion for Interpreter.

2016-07-14 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1117 @bzz I totally agree with you so I did try to remove things you mentioned but it's not easy. Let me try one more. Thanks! --- If your project is set up for it, you can reply to this email

[GitHub] zeppelin issue #1117: [ZEPPELIN-1086] Auto Completion for Interpreter.

2016-07-15 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1117 @bzz Thank you kindly for telling like that. What do you think about tagging meta value than removing keywords like following image? ![image](https://cloud.githubusercontent.com

[GitHub] zeppelin issue #1191: [ZEPPELIN-1186] Prevent NPE when unbind interpreter

2016-07-17 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1191 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 issue #1193: [ZEPPELIN-1192] Block pyspark paragraph hang.

2016-07-19 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1193 please 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 your project does not have this feature enabled

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

2016-07-15 Thread astroshim
GitHub user astroshim opened 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

[GitHub] zeppelin issue #1196: [ZEPPELIN-1195] Unbind interpreter error.

2016-07-16 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1196 Oh sorry. --- 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 #1185: Redirect home when requested a wrong notename p...

2016-07-14 Thread astroshim
GitHub user astroshim reopened a pull request: https://github.com/apache/zeppelin/pull/1185 Redirect home when requested a wrong notename path. ### What is this PR for? This PR for redirecting to zeppelin home when requested wrong path. ### What type of PR

[GitHub] zeppelin issue #1090: [MINOR] Blocking the minus cursor value.

2016-06-27 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1090 Thank you for your review! @bzz --- 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 #1030: [ZEPPELIN-1023] Add more credential apis.

2016-06-27 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1030 @bzz okay I will. 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 this feature enabled

[GitHub] zeppelin issue #1097: [Bugfix] init RepoSetting values

2016-06-28 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1097 It seems that the CI error is not related this PR. ``` Tests in error: ZeppelinIT.testSparkInterpreterDependencyLoading:217->AbstractZeppelinIT.createNewNote:119 » Time

[GitHub] zeppelin issue #1100: [ZEPPELIN-1054] Improve "Credentials" UI

2016-06-28 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1100 Really great feature! Thanks for making 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 have

[GitHub] zeppelin issue #1099: ZEPPELIN-1071 ] Ace-editor hidden auto-complete additi...

2016-06-28 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1099 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 #1104: [ZEPPELIN-1078] Does not show the text result i...

2016-06-29 Thread astroshim
Github user astroshim closed the pull request at: https://github.com/apache/zeppelin/pull/1104 --- 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 #1104: [ZEPPELIN-1078] Does not show the text result i...

2016-06-29 Thread astroshim
GitHub user astroshim reopened a pull request: https://github.com/apache/zeppelin/pull/1104 [ZEPPELIN-1078] Does not show the text result in the paragraph sometimes. ### What is this PR for? This PR fixes the problem that does not show the text result in the paragraph sometimes

[GitHub] zeppelin pull request #1097: [Bugfix] init RepoSetting values

2016-06-28 Thread astroshim
GitHub user astroshim reopened a pull request: https://github.com/apache/zeppelin/pull/1097 [Bugfix] init RepoSetting values ### What is this PR for? This PR fixes initialize value bug of ```Add New Repository```. ### What type of PR is it? Bug Fix

[GitHub] zeppelin pull request #1097: [Bugfix] init RepoSetting values

2016-06-28 Thread astroshim
Github user astroshim closed the pull request at: https://github.com/apache/zeppelin/pull/1097 --- 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 #1097: [Bugfix] init RepoSetting values

2016-06-29 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1097 CI has passed. 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 #1097: [Bugfix] init RepoSetting values

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