[GitHub] incubator-zeppelin pull request: [ZEPPELIN-502] Python interpreter...

2016-05-21 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/869#issuecomment-220805293 This looks great, thanks, just a couple of minor comments. --- If your project is set up for it, you can reply to this email and have your reply appear o

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

2016-05-21 Thread prasadwagle
Github user prasadwagle commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/860#issuecomment-220805129 @Leemoonsoo I agree. I removed HiveInterpreter changes from this PR. What do you think about using Entity instead of Data source? --- If your project is

[GitHub] incubator-zeppelin pull request: ZEPPELIN-277: Set the (TAB) key a...

2016-05-20 Thread vinayshukla
Github user vinayshukla commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/273#issuecomment-220756425 @tzolov Thanks for your work on this. Are you still working on this? What is needed to complete this one? --- If your project is set up for it, you can

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

2016-05-20 Thread bzz
Github user bzz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/880#issuecomment-220689981 Huge 👍 for tests and test notebooks. Looks great to me modulo minor comments above. --- If your project is set up for it, you can reply to this emai

[GitHub] incubator-zeppelin pull request: Update/shiro docs

2016-05-20 Thread vinayshukla
Github user vinayshukla commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/907#issuecomment-220663820 @AhyoungRyu Thanks for this change. I am happy to see this doc improvement. But I think we have an issue with authentication performed by upstream

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

2016-05-20 Thread natesammons-nasdaq
Github user natesammons-nasdaq commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/886#issuecomment-220608371 Is there anything else I need to do from my end? Thanks --- If your project is set up for it, you can reply to this email and have your reply app

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

[GitHub] incubator-zeppelin pull request: Update/shiro docs

2016-05-20 Thread AhyoungRyu
GitHub user AhyoungRyu opened a pull request: https://github.com/apache/incubator-zeppelin/pull/907 Update/shiro docs ### What is this PR for? Currently, Zeppelin have two authentication docs. One is [**Authentication**](https://zeppelin.incubator.apache.org/docs/0.6.0-incubati

[GitHub] incubator-zeppelin pull request: Add _site dir to .gitignore file

2016-05-20 Thread AhyoungRyu
GitHub user AhyoungRyu opened a pull request: https://github.com/apache/incubator-zeppelin/pull/906 Add _site dir to .gitignore file ### What is this PR for? When we execute Jekyll to build Zeppelin docs, `_site` directory is generated automatically. So I added `_site` to `.giti

[GitHub] incubator-zeppelin pull request: ZEPPELIN-859 Error on notebook pa...

2016-05-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/896 --- 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 fea

[GitHub] incubator-zeppelin pull request: ZEPPELIN-858: make interpreter se...

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

[GitHub] incubator-zeppelin pull request: Update HttpClient Dependencies

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

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

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

[GitHub] incubator-zeppelin pull request: Sjk dev

2016-05-19 Thread shijinkui
GitHub user shijinkui opened a pull request: https://github.com/apache/incubator-zeppelin/pull/905 Sjk dev ### What is this PR for? 1. frontend-maven-plugin upgrade to 1.0, as low version occur error when build on osx 2. upgrade spark version from 1.4.1 to 1.6.1 3. defin

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

2016-05-19 Thread AhyoungRyu
Github user AhyoungRyu commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/880#issuecomment-220502696 @khalidhuseynov Great work indeed! Why don't you add the below line to [here](https://github.com/apache/incubator-zeppelin/blob/master/docs/_includes/

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

2016-05-19 Thread AhyoungRyu
Github user AhyoungRyu commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/901#issuecomment-220497669 @r-kamath Right that would be also a possible way :) But then the tooltip will hide `+Create` button, so I just switched them. --- If your project is set

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

2016-05-19 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/858#issuecomment-220496295 @yantaiv Better ask that in the mailing list, you would get more answers But overall: You can use any IDE you want, there is informations about dev

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

2016-05-19 Thread yantaiv
Github user yantaiv commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/858#issuecomment-220495083 Great Job. I have a silly question, how do you debug your code in zeppelin, I mean , what IDE do you use and how you make breakpoint or sth like that ... -

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

2016-05-19 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/860#issuecomment-220488582 @prasadwagle Tried and looks good. I think separate issue can handle address HiveInterpreter changes. If you'd prefer. One thing is, credential he

Re: [GitHub] incubator-zeppelin pull request: [ZEPPELIN-577] ASCII control char...

2016-05-19 Thread Brian Cohn
unsubscribe On Tue, May 17, 2016 8:56 PM, minahlee g...@git.apache.org wrote: GitHub user minahlee opened a pull request: https://github.com/apache/incubator-zeppelin/pull/894 [ZEPPELIN-577] ASCII control character errors in Zeppelin ### What is this PR for? Take care of `interpr

[GitHub] incubator-zeppelin pull request: Fix interpreter.sh classpath

2016-05-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/889 --- 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 fea

[GitHub] incubator-zeppelin pull request: Fix link to r interpreter doc

2016-05-19 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/898#issuecomment-220413194 Merging into master as a hotfix. --- 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 p

[GitHub] incubator-zeppelin pull request: Fix link to r interpreter doc

2016-05-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/898 --- 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 fea

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-577] ASCII control char...

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

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

2016-05-19 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/901#issuecomment-220338513 @AhyoungRyu If tooltip display is the only issue you can try other placement properties `tooltip-placement="left"` otherwise LGTM 👍 -

[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 it? Bug Fix

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-773] Adding support for...

2016-05-19 Thread mfelgamal
GitHub user mfelgamal opened a pull request: https://github.com/apache/incubator-zeppelin/pull/903 [ZEPPELIN-773] Adding support for spark interpreter to work with yarn… ### What is this PR for? Adding support for zeppelin to run in yarn-cluster mode through Livy - Creating

[GitHub] incubator-zeppelin pull request: ZEPPELIN-859 Error on notebook pa...

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

[GitHub] incubator-zeppelin pull request: Documentation for setting Azure n...

2016-05-19 Thread khalidhuseynov
GitHub user khalidhuseynov opened a pull request: https://github.com/apache/incubator-zeppelin/pull/902 Documentation for setting Azure notebook storage ### What is this PR for? This PR adds general info and documentation on setting Azure storage in the `docs/storage.md` folder

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

2016-05-19 Thread AhyoungRyu
GitHub user AhyoungRyu opened a pull request: https://github.com/apache/incubator-zeppelin/pull/901 Change the placement of 'Repository Information' icon with '+Create' button ### What is this PR for? I just changed the `Repository Information` icon placement with `+Create` butt

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

2016-05-19 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/827#issuecomment-220255952 Thank you @AhyoungRyu for the review. Have addressed those. --- If your project is set up for it, you can reply to this email and have your reply app

[GitHub] incubator-zeppelin pull request: ZEPPELIN-858: make interpreter se...

2016-05-19 Thread AhyoungRyu
Github user AhyoungRyu commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/895#issuecomment-220247660 @prabhjyotsingh Really useful feature! It works well. --- If your project is set up for it, you can reply to this email and have your reply appear on GitH

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-577] ASCII control char...

2016-05-19 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/894#issuecomment-220245108 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 fe

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-789] Skip execution of ...

2016-05-18 Thread springml
Github user springml closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/812 --- 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] incubator-zeppelin pull request: [ZEPPELIN-789] Skip execution of ...

2016-05-18 Thread springml
GitHub user springml reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/812 [ZEPPELIN-789] Skip execution of a paragraph in case of error ### What is this PR for? New option in paragraphs to skip execution, when notebook is executed as a whole and there

[GitHub] incubator-zeppelin pull request: Fixing Zeppelin-838: Minor improv...

2016-05-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/872 --- 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 fea

[GitHub] incubator-zeppelin pull request: ZEPPELIN-858: make interpreter se...

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

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-577] ASCII control char...

2016-05-18 Thread minahlee
GitHub user minahlee reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/894 [ZEPPELIN-577] ASCII control character errors in Zeppelin ### What is this PR for? Take care of `interpreter not found` issue when code is copied from CRLF line ending editor

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-577] ASCII control char...

2016-05-18 Thread minahlee
Github user minahlee closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/894 --- 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] incubator-zeppelin pull request: [ZEPPELIN-789] Skip execution of ...

2016-05-18 Thread khalidhuseynov
Github user khalidhuseynov commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/812#issuecomment-220217140 Could you reopen PR to trigger CI? The build failure doesn't seem to be related --- If your project is set up for it, you can reply to this email and

[GitHub] incubator-zeppelin pull request: Make zeppelin work with CDH5.7.0

2016-05-18 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/868#issuecomment-220216617 @secsubs unfortunately no, it seems to be a breaking API change in CDH-5.7 (also in Spark 2.0) Let's get Travis tests to pass and merge this ASAP. Tha

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-577] ASCII control char...

2016-05-18 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/894#issuecomment-220215547 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 f

[GitHub] incubator-zeppelin pull request: Refactor CORS filter into webapp ...

2016-05-18 Thread khalidhuseynov
GitHub user khalidhuseynov opened a pull request: https://github.com/apache/incubator-zeppelin/pull/900 Refactor CORS filter into webapp context handler ### What is this PR for? #867 was a hotfix and merged immediately. However we can refactor CORS filter into more appropriate p

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

2016-05-18 Thread natesammons-nasdaq
Github user natesammons-nasdaq commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/886#issuecomment-220214238 Looks like the last build passed --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. I

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

2016-05-18 Thread natesammons-nasdaq
Github user natesammons-nasdaq closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/886 --- 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, o

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

2016-05-18 Thread natesammons-nasdaq
GitHub user natesammons-nasdaq reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/886 [ZEPPELIN-848] Add support for encrypted data stored in Amazon S3 ### What is this PR for? Adds support for using the AWS KMS or a custom encryption materials provide

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

2016-05-18 Thread natesammons-nasdaq
Github user natesammons-nasdaq commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/886#issuecomment-220208525 I looked at the latest failure, not sure what the cause is. Looks like: Tests run: 6, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 81

[GitHub] incubator-zeppelin pull request: Fix link to r interpreter doc

2016-05-18 Thread AhyoungRyu
Github user AhyoungRyu commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/898#issuecomment-220205263 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 t

[GitHub] incubator-zeppelin pull request: Fix a few image not rendering

2016-05-18 Thread corneadoug
GitHub user corneadoug opened a pull request: https://github.com/apache/incubator-zeppelin/pull/899 Fix a few image not rendering ### What is this PR for? There was a recent PR made by @doanduyhai that was changing some images URL to include `../assets`. However, I found a few m

[GitHub] incubator-zeppelin pull request: Make zeppelin work with CDH5.7.0

2016-05-18 Thread secsubs
Github user secsubs commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/868#issuecomment-220170793 I have the exact same issue with CDH-5.7.0 and Zepplin-0.5.6. Is there a workaround available? Thanks. --- If your project is set up for it, you can reply t

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

2016-05-18 Thread natesammons-nasdaq
Github user natesammons-nasdaq commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/886#issuecomment-220169618 Merged changes to trigger another build --- 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-848] Add support for en...

2016-05-18 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/886#issuecomment-220167056 I think there were some problem on "repository.apache.org" and i believe now it is stable. @natesammons-nasdaq Could you try once again re-trigger the CI?

[GitHub] incubator-zeppelin pull request: Fix interpreter.sh classpath

2016-05-18 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/889#issuecomment-220165691 Every jar under interpreter/spark/ supposed to be included in the classpath by [interpreter.sh#67](https://github.com/apache/incubator-zeppelin/blob/maste

[GitHub] incubator-zeppelin pull request: Fix notebook folder structure nam...

2016-05-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/883 --- 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 fea

[GitHub] incubator-zeppelin pull request: Revise documentation about basic ...

2016-05-18 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/877 --- 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 fea

[GitHub] incubator-zeppelin pull request: Fix link to r interpreter doc

2016-05-18 Thread Leemoonsoo
GitHub user Leemoonsoo opened a pull request: https://github.com/apache/incubator-zeppelin/pull/898 Fix link to r interpreter doc ### What is this PR for? This PR is hot fix for broken link to r interpreter doc ### What type of PR is it? Hot Fix ### Tod

[GitHub] incubator-zeppelin pull request: Fixing Zeppelin-838: Minor improv...

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

[GitHub] incubator-zeppelin pull request: Update HttpClient Dependencies

2016-05-18 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/897#issuecomment-220091243 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 feat

[GitHub] incubator-zeppelin pull request: Fixing Zeppelin-838: Minor improv...

2016-05-18 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/872#issuecomment-220091216 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 feat

[GitHub] incubator-zeppelin pull request: Update HttpClient Dependencies

2016-05-18 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/897#issuecomment-220088990 CI is 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

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

2016-05-18 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/827#issuecomment-220088767 @Leemoonsoo thanks a lot for reviewing it. CI fails for unrelated issue at Job #4237.1 --- If your project is set up for it, you can reply to

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

2016-05-18 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/827#issuecomment-220086349 @prabhjyotsingh Thanks for the help. Commenting out spark.master from https://github.com/cloudera/livy/conf/spark-blacklist.conf made it work. Loo

[GitHub] incubator-zeppelin pull request: Update HttpClient Dependencies

2016-05-18 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request: https://github.com/apache/incubator-zeppelin/pull/897 Update HttpClient Dependencies ### What is this PR for? HttpClient dependencies of the stack needs to be addressed as below (due to end of life of commons-httpclient and secur

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

2016-05-18 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/827#issuecomment-220059254 @Leemoonsoo can you try one of this `curl -X POST --data '{"kind": "spark", "master": "local[*]", "proxyUser": "null"}' -H "Content-Type: appl

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

2016-05-18 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/827#issuecomment-220023617 @prabhjyotsingh Thanks for explanation. I have tried with https://github.com/cloudera/hue/tree/master/apps/spark/java and LivyInterpreter works greatly.

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

2016-05-18 Thread khalidhuseynov
Github user khalidhuseynov commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/880#issuecomment-219983068 This is 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 proje

[GitHub] incubator-zeppelin pull request: ZEPPELIN-859 Error on notebook pa...

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

[GitHub] incubator-zeppelin pull request: ZEPPELIN-859 Error on notebook pa...

2016-05-18 Thread r-kamath
GitHub user r-kamath opened a pull request: https://github.com/apache/incubator-zeppelin/pull/896 ZEPPELIN-859 Error on notebook page due null property ### What is this PR for? Add null check to prevent js error on notebook page ### What type of PR is it? Bug Fix

[GitHub] incubator-zeppelin pull request: import org.slf4j instead of parqu...

2016-05-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/892 --- 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 fea

[GitHub] incubator-zeppelin pull request: merge:from apache-master(for zepp...

2016-05-17 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/893#issuecomment-219926200 did you open this by accident? please close this? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] incubator-zeppelin pull request: ZEPPELIN-858: make interpreter se...

2016-05-17 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request: https://github.com/apache/incubator-zeppelin/pull/895 ZEPPELIN-858: make interpreter searchable and sorted ### What is this PR for? Interpreters listed `/#/interpreter` here are slightly difficult to find, also the list that show

[GitHub] incubator-zeppelin pull request: ZEPPELIN-850 update error message

2016-05-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/888 --- 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 fea

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-577] ASCII control char...

2016-05-17 Thread minahlee
Github user minahlee closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/894 --- 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] incubator-zeppelin pull request: [ZEPPELIN-577] ASCII control char...

2016-05-17 Thread minahlee
GitHub user minahlee reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/894 [ZEPPELIN-577] ASCII control character errors in Zeppelin ### What is this PR for? Take care of `interpreter not found` issue when code is copied from CRLF line ending editor

[GitHub] incubator-zeppelin pull request: Fix notebook folder structure nam...

2016-05-17 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/883#issuecomment-219919689 Merge it into master if there're no more discussions --- If your project is set up for it, you can reply to this email and have your reply appear on GitHu

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-577] ASCII control char...

2016-05-17 Thread minahlee
GitHub user minahlee opened a pull request: https://github.com/apache/incubator-zeppelin/pull/894 [ZEPPELIN-577] ASCII control character errors in Zeppelin ### What is this PR for? Take care of `interpreter not found` issue when code is copied from CRLF line ending editor

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-281] Windows build

2016-05-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/876 --- 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 fea

[GitHub] incubator-zeppelin pull request: merge:from apache-master(for zepp...

2016-05-17 Thread hj5
Github user hj5 commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/893#issuecomment-219916185 zeppelin升级改造 --- 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 hav

[GitHub] incubator-zeppelin pull request: merge:from apache-master(for zepp...

2016-05-17 Thread hj5
GitHub user hj5 opened a pull request: https://github.com/apache/incubator-zeppelin/pull/893 merge:from apache-master(for zeppelin 0.5.6) ### What is this PR for? A few sentences describing the overall goals of the pull request's commits. First time? Check out the contributin

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

2016-05-17 Thread natesammons-nasdaq
Github user natesammons-nasdaq commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/886#issuecomment-219914133 Build is still failing with the following: [ERROR] Failed to execute goal org.apache.maven.plugins:maven-remote-resources-plugin:1.4:proce

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

2016-05-17 Thread natesammons-nasdaq
Github user natesammons-nasdaq commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/886#issuecomment-219913297 Re-opening to 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 p

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

2016-05-17 Thread natesammons-nasdaq
GitHub user natesammons-nasdaq reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/886 [ZEPPELIN-848] Add support for encrypted data stored in Amazon S3 ### What is this PR for? Adds support for using the AWS KMS or a custom encryption materials provide

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

2016-05-17 Thread natesammons-nasdaq
Github user natesammons-nasdaq closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/886 --- 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, o

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

2016-05-17 Thread natesammons-nasdaq
Github user natesammons-nasdaq commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/886#issuecomment-219913275 Closing request to re-open 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 y

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

2016-05-17 Thread natesammons-nasdaq
Github user natesammons-nasdaq commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/886#issuecomment-219912714 Sure, I'll try that. I haven't been able to see any build status yet, they all appear to fail with a timeout under Travis. ? Sent from m

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

2016-05-17 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/886#issuecomment-219908546 While [all build profile failed](https://travis-ci.org/apache/incubator-zeppelin/builds/130849266) for last commit with some network issue, It's good idea

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

2016-05-17 Thread khalidhuseynov
Github user khalidhuseynov commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/886#issuecomment-219900529 thanks for handling exceptions, LGTM. may need to re-trigger CI, although the build failure isn't related at all. --- If your project is set up for

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

[GitHub] incubator-zeppelin pull request: Make zeppelin work with CDH5.7.0

2016-05-17 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/868#issuecomment-219814326 Thanks!  CDH is known to have an older Akka and normally we build with the profile -Pvendor-repo to get that to work.

[GitHub] incubator-zeppelin pull request: ZEPPELIN-850 update error message

2016-05-17 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/888#issuecomment-219770129 CI fails at `Job #4165.6` for `npm install --color=false` which is unrelated. Merging this if no more discussion. --- If your project is set

[GitHub] incubator-zeppelin pull request: Make zeppelin work with CDH5.7.0

2016-05-17 Thread krisgeus
Github user krisgeus commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/868#issuecomment-219762689 @felixcheung I've made some progress in the testing area. It appears that spark-core-1.6.0-cdh5.7.0 brings in older akka version 2.2.3 instead of the 2.3.11

[GitHub] incubator-zeppelin pull request: Revise documentation about basic ...

2016-05-17 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/877#issuecomment-219736646 Thanks @yoonjs2 for the nice improvement on documentation. LGTM and merge if there're no more discussions! --- If your project is set up for it, you c

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

2016-05-17 Thread natesammons-nasdaq
Github user natesammons-nasdaq commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/886#issuecomment-219734504 Sorry about that! --- 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: import org.slf4j instead of parqu...

2016-05-17 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/892#issuecomment-219733633 @prabhjyotsingh Thanks for taking a look. CI failure is unrelated. Merge into master if there're no more discussions --- If your project is set up

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-185] ZeppelinContext me...

2016-05-17 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/176#issuecomment-219712754 I don't think this is currently supported, looking at the code. The error is caused by py4j trying to convert the DataFrame type. Do you mind openi

[GitHub] incubator-zeppelin pull request: Make zeppelin work with CDH5.7.0

2016-05-17 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/868#issuecomment-219712068 Does the fix in this PR make it work for your case? --- If your project is set up for it, you can reply to this email

[GitHub] incubator-zeppelin pull request: import org.slf4j instead of parqu...

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

[GitHub] incubator-zeppelin pull request: ZEPPELIN-852 fixed the fields flo...

2016-05-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/890 --- 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 fea

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

2016-05-17 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/827#issuecomment-219664223 Thank you @Leemoonsoo for the review, on debugging I found out that the Livy-Server that https://github.com/cloudera/livy and https://github.com/cloud

[GitHub] incubator-zeppelin pull request: Make zeppelin work with CDH5.7.0

2016-05-17 Thread H4ml3t
Github user H4ml3t commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/868#issuecomment-219653465 @felixcheung not much to elaborate, I would need zeppelin to work with external Spark installation with cdh5.7 :) Cheers, Luca --- If your project is

  1   2   3   4   5   6   7   8   9   10   >