[GitHub] zeppelin issue #1059: [ZEPPELIN-954] Fix table cell selection problem on sec...

2016-06-24 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1059 Merge it there is no more discussion --- 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 ena

[GitHub] zeppelin pull request #1066: [ZEPPELIN-1047] Add filter to handle upper case

2016-06-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1066 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is e

[GitHub] zeppelin pull request #971: [ZEPPELIN-961] Longer names to the notebook make...

2016-06-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/971 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is en

[GitHub] zeppelin issue #1084: [HOTFIX] Fix flaky AuthenticationIT

2016-06-24 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1084 @minahlee @bzz @Leemoonsoo After merging this CI is now failing (Build #5323.1) with ``` --- T E S T S ---

[GitHub] zeppelin pull request #1084: [HOTFIX] Fix flaky AuthenticationIT

2016-06-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1084 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is e

[GitHub] zeppelin issue #1084: [HOTFIX] Fix flaky AuthenticationIT

2016-06-24 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1084 Thanks a lot, I'll merge this as a hot fix. --- 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 thi

[GitHub] zeppelin issue #1084: [HOTFIX] Fix flaky AuthenticationIT

2016-06-24 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1084 Awesome!!! let's merge 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 wi

[GitHub] zeppelin issue #1084: [HOTFIX] Fix flaky AuthenticationIT

2016-06-24 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1084 CI green ready for 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 this feature enabled

[GitHub] zeppelin issue #1081: [ZEPPELIN-846] Fix Long Notebook Dropdown

2016-06-24 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1081 @Leemoonsoo Just fixed that and a few HTML problems when minified. Jshint didn't run during dev mode which is a bit fishy. --- If your project is set up for it, you can reply to this email

[GitHub] zeppelin pull request #1087: Shell intepreter test and doc

2016-06-24 Thread fvaleri
GitHub user fvaleri opened a pull request: https://github.com/apache/zeppelin/pull/1087 Shell intepreter test and doc ### What is this PR for? Small refactoring to Shell interpreter. Add missing unit test and initial doc. ### What type of PR is it? Improvement, D

[GitHub] zeppelin pull request #1086: [Zeppelin 1042] Extra space is present as part ...

2016-06-24 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request: https://github.com/apache/zeppelin/pull/1086 [Zeppelin 1042] Extra space is present as part of username in search box ### What is this PR for? Sometimes extra space is present as part of username in search box while trying to set

[GitHub] zeppelin issue #1071: [ZEPPELIN-1052] Application does not logout user when ...

2016-06-24 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1071 @minahlee Thank you for much explanation about the issue.Have open up https://github.com/apache/zeppelin/pull/1084 to find and fix this issue, I'll try to resolve ASAP. --- If your projec

[GitHub] zeppelin issue #1081: [ZEPPELIN-846] Fix Long Notebook Dropdown

2016-06-24 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1081 @corneadoug Tested and works well. Further work doesn't need to be addressed in this PR I think. By the way, CI build is failing with error ``` INFO] src/components/expandC

[GitHub] zeppelin issue #1085: [MINOR]Removed some codes using interpreterClassList

2016-06-24 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1085 @jongyoul Yes, i thought so. --- 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 a

[GitHub] zeppelin issue #1085: [MINOR]Removed some codes using interpreterClassList

2016-06-24 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1085 @Leemoonsoo I've figured out what you said. This PR changes the way to find a default interpreter, isn't it? --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] zeppelin issue #1085: [MINOR]Removed some codes using interpreterClassList

2016-06-24 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1085 @Leemoonsoo Could you please describe in details? Do you mean the order of interpreter tab? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHu

[GitHub] zeppelin issue #1077: [ZEPPELIN-921] Apply new mechanism to KnitR and RRepl

2016-06-24 Thread rerngvit
Github user rerngvit commented on the issue: https://github.com/apache/zeppelin/pull/1077 @jongyoul regarding the description text, according to the [build documentation](https://github.com/apache/zeppelin#build), both options should enable R support. Please let me know, if I miss som

[GitHub] zeppelin issue #1085: [MINOR]Removed some codes using interpreterClassList

2016-06-24 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1085 Iterating interpreterClassList is mainly for keeping order of interpreters defined in `zeppelin.interpreters` property. Therefore this PR will not only refactoring the code but also bri

[GitHub] zeppelin pull request #1085: [MINOR]Removed some codes using interpreterClas...

2016-06-24 Thread jongyoul
GitHub user jongyoul opened a pull request: https://github.com/apache/zeppelin/pull/1085 [MINOR]Removed some codes using interpreterClassList ### What is this PR for? Removing dependency from interpreterClassList because this prevent new interpreter without interpreterClassList

[GitHub] zeppelin pull request #1072: [MINOR] Remove unused util methods and its test...

2016-06-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1072 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is e

[GitHub] zeppelin pull request #1083: [WIF] Fix flaky AuthenticationIT

2016-06-24 Thread prabhjyotsingh
Github user prabhjyotsingh closed the pull request at: https://github.com/apache/zeppelin/pull/1083 --- 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 feat

[GitHub] zeppelin pull request #1084: [WIF] Fix flaky AuthenticationIT

2016-06-24 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request: https://github.com/apache/zeppelin/pull/1084 [WIF] Fix flaky AuthenticationIT ### What is this PR for? Tests in error: AuthenticationIT.testGroupPermission:177->authenticationUser:109->AbstractZeppelinIT.pollingWait:114 Ã

[GitHub] zeppelin issue #1071: [ZEPPELIN-1052] Application does not logout user when ...

2016-06-24 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1071 @prabhjyotsingh Thanks for quick response. The steps I wrote was just for reproducing the case that logout action doesn't work because I couldn't reproduce it in my laptop with shiro setting in `

[GitHub] zeppelin pull request #1083: [WIF] Fix flaky AuthenticationIT

2016-06-24 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request: https://github.com/apache/zeppelin/pull/1083 [WIF] Fix flaky AuthenticationIT ### What is this PR for? Tests in error: AuthenticationIT.testGroupPermission:177->authenticationUser:109->AbstractZeppelinIT.pollingWait:114 Ã

[GitHub] zeppelin issue #1071: [ZEPPELIN-1052] Application does not logout user when ...

2016-06-24 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/1071 @minahlee, Thank you for review this. With default shiro.ini i.e. ``` /api/version = anon /** = anon #/** = authc ``` Under `Insufficient privileges

[GitHub] zeppelin issue #971: [ZEPPELIN-961] Longer names to the notebook make the na...

2016-06-24 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/971 LGTM merging it into master and branch-0.6 --- 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 featur

[GitHub] zeppelin issue #1081: [ZEPPELIN-846] Fix Long Notebook Dropdown

2016-06-24 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1081 @corneadoug Awesome! I tested and it works really well 👍 --- 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 doe

[GitHub] zeppelin issue #1071: [ZEPPELIN-1052] Application does not logout user when ...

2016-06-24 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1071 @prabhjyotsingh Thanks for the fix! Seems like this PR is bringing CI error in master, branch-0.6: ``` Tests run: 1, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 81.986 sec <<< FA

[GitHub] zeppelin pull request #1082: [ZEPPELIN-1056] Embed webfont when building

2016-06-24 Thread corneadoug
GitHub user corneadoug opened a pull request: https://github.com/apache/zeppelin/pull/1082 [ZEPPELIN-1056] Embed webfont when building ### What is this PR for? This is some changes to include the previously webfonts when building Zeppelin. If there is too much problem buildi

[GitHub] zeppelin issue #1066: [ZEPPELIN-1047] Add filter to handle upper case

2016-06-24 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1066 Merge if there is no more dicussion --- 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 enab

[GitHub] zeppelin pull request #1081: [ZEPPELIN-846] Fix Long Notebook Dropdown

2016-06-24 Thread corneadoug
GitHub user corneadoug opened a pull request: https://github.com/apache/zeppelin/pull/1081 [ZEPPELIN-846] Fix Long Notebook Dropdown ### What is this PR for? This PR is fixing the bug by implementing a new layout for the Notebook Dropdown menu. Reason being that setting a ma

[GitHub] zeppelin pull request #1080: [ZEPPELIN-1060] validate user before saving

2016-06-24 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request: https://github.com/apache/zeppelin/pull/1080 [ZEPPELIN-1060] validate user before saving ### What is this PR for? Zeppelin notebook permissions change has an issue while reconfiguring permissions by clearing textbox ###

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

2016-06-24 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1079 Better! 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

[jira] [Created] (ZEPPELIN-1060) Zeppelin notebook permissions change has an issue while reconfiguring permissions by clearing textbox

2016-06-24 Thread Prabhjyot Singh (JIRA)
Prabhjyot Singh created ZEPPELIN-1060: - Summary: Zeppelin notebook permissions change has an issue while reconfiguring permissions by clearing textbox Key: ZEPPELIN-1060 URL: https://issues.apache.org/jira/bro

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

2016-06-24 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1030 @bzz @AhyoungRyu Could you 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 this feature enab

[GitHub] zeppelin pull request #1055: [ZEPPELIN-1028] Fix exported notebook importing...

2016-06-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1055 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is e

[GitHub] zeppelin issue #1072: [MINOR] Remove unused util methods and its tests

2016-06-24 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1072 Merging it into master and branch-0.6 --- 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 en

[jira] [Created] (ZEPPELIN-1059) Notebook title shoudn't appear in the print result

2016-06-24 Thread JIRA
François Rosière created ZEPPELIN-1059: -- Summary: Notebook title shoudn't appear in the print result Key: ZEPPELIN-1059 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1059 Project: Zeppelin

[GitHub] zeppelin issue #1066: [ZEPPELIN-1047] Add filter to handle upper case

2016-06-24 Thread suvam97
Github user suvam97 commented on the issue: https://github.com/apache/zeppelin/pull/1066 @minahlee Conflict Resolved --- 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 #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, or if the f

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

2016-06-24 Thread prabhjyotsingh
Github user prabhjyotsingh 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, or if

[GitHub] zeppelin issue #1077: [ZEPPELIN-921] Apply new mechanism to KnitR and RRepl

2016-06-24 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1077 @rerngvit In a description, I think `-Psparkr` is not related to this module. we should use `-Pr`, right? --- If your project is set up for it, you can reply to this email and have your reply ap

[GitHub] zeppelin pull request #1079: Prevent menubar hide notebook title and control...

2016-06-24 Thread Leemoonsoo
GitHub user Leemoonsoo opened a pull request: https://github.com/apache/zeppelin/pull/1079 Prevent menubar hide notebook title and control buttons depending on screen size ### What is this PR for? Prevent menubar hide notebook title and control buttons, depending on screen size

[GitHub] zeppelin pull request #1049: [ZEPPELIN-1034] Add Spark Interpreter option to...

2016-06-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1049 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is e

[GitHub] zeppelin pull request #1071: [ZEPPELIN-1052] Application does not logout use...

2016-06-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1071 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is e

[GitHub] zeppelin pull request #1078: [ZEPPELIN-936] Fix flaky test SparkRTest

2016-06-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1078 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is e

[GitHub] zeppelin issue #1078: [ZEPPELIN-936] Fix flaky test SparkRTest

2016-06-24 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1078 I think the reason SparkRTest() failing was because of resource. pySpark() test creates 33 python processes (num core + 1) and that seems make unable to run SparkRTest(). I have cha

[GitHub] zeppelin issue #971: [ZEPPELIN-961] Longer names to the notebook make the na...

2016-06-24 Thread sagarkulkarni3592
Github user sagarkulkarni3592 commented on the issue: https://github.com/apache/zeppelin/pull/971 @minahlee Also merged it with the master branch. --- 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 h