[GitHub] incubator-zeppelin pull request: [ZEPPELIN-848] Add support for en...

2016-05-23 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/886#issuecomment-221173891 Tested and working well for me. One more thing, could you add new env variables and properties to

[GitHub] incubator-zeppelin pull request: Change the placement of 'Reposito...

2016-05-23 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/901#issuecomment-221169107 Merging this if 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.

[GitHub] incubator-zeppelin pull request: move noteName method from main to...

2016-05-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/910 --- 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: Update/shiro docs

2016-05-23 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/907#issuecomment-221167326 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

[GitHub] incubator-zeppelin pull request: ZEPPELIN-773 : Livy interperter

2016-05-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/827 --- 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: Add _site dir to .gitignore file

2016-05-23 Thread bzz
Github user bzz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/906#issuecomment-221155026 👍 merging if 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

[GitHub] incubator-zeppelin pull request: ZEPPELIN-866] Hotfix - does not w...

2016-05-23 Thread cloverhearts
Github user cloverhearts commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/911#issuecomment-221152608 @astroshim Thank you for feed back. are you mean 'initial statement' is 'clean up on search box'? --- If your project is set up for it, you can

[GitHub] incubator-zeppelin pull request: ZEPPELIN-866] Hotfix - does not w...

2016-05-23 Thread bzz
Github user bzz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/911#issuecomment-221151213 Thanks for hotfix, but do we know why it stopped working in the first place? Let me take a look. \cc @felizbear for review as well --- If your

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

2016-05-23 Thread mwkang
Github user mwkang commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/904#issuecomment-221146929 @Leemoonsoo Part of Ci are failed. Because.. [ERROR] Failed to execute goal on project zeppelin-zrinterpreter: Could not resolve dependencies

[GitHub] incubator-zeppelin pull request: ZEPPELIN-866] Hotfix - does not w...

2016-05-23 Thread cloverhearts
Github user cloverhearts commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/911#issuecomment-221147514 @prabhjyotsingh fix done. Please, Can you feedback for this pr for me? --- If your project is set up for it, you can reply to this email and

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

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

2016-05-23 Thread mwkang
GitHub user mwkang reopened 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 it? Bug

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

2016-05-23 Thread mwkang
Github user mwkang commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/904#issuecomment-221143687 Of course. I close the PR and re-open to re-trigger the ci. --- 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: [ZEPPELIN-835]Don't persist authe...

2016-05-23 Thread mwkang
Github user mwkang closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/904 --- 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

[jira] [Created] (ZEPPELIN-868) Notebook import fails when notebook is large

2016-05-23 Thread Sofia Heisler (JIRA)
Sofia Heisler created ZEPPELIN-868: -- Summary: Notebook import fails when notebook is large Key: ZEPPELIN-868 URL: https://issues.apache.org/jira/browse/ZEPPELIN-868 Project: Zeppelin Issue

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-828] Use individual use...

2016-05-23 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/860#issuecomment-221030666 Thanks @prasadwagle. It looks good to me! --- 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-773 : Livy interperter

2016-05-23 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/827#issuecomment-221025863 +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

[jira] [Created] (ZEPPELIN-867) did not see the Dependency Management for Interpreter in the latest build

2016-05-23 Thread chong chow (JIRA)
chong chow created ZEPPELIN-867: --- Summary: did not see the Dependency Management for Interpreter in the latest build Key: ZEPPELIN-867 URL: https://issues.apache.org/jira/browse/ZEPPELIN-867 Project:

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-704] Display elapse tim...

2016-05-23 Thread sagarkulkarni3592
GitHub user sagarkulkarni3592 reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/862 [ZEPPELIN-704] Display elapse time for long running paragraph ### What is this PR for? For long running paragraph, it is hard to know how long it has already been

[GitHub] incubator-zeppelin pull request: ZEPPELIN-866] Hotfix - does not w...

2016-05-23 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/911#issuecomment-220962020 Or that is also broken for `$locationChangeSuccess`, its not cleaning up. Can you fix this as well ? --- If your project is set up for it, you can

[GitHub] incubator-zeppelin pull request: ZEPPELIN-866] Hotfix - does not w...

2016-05-23 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/911#issuecomment-220959983 If we use `$scope.search = function(searchTerm)` then we can