[GitHub] incubator-zeppelin pull request: Pass defaultValue parameter to z....

2016-01-12 Thread corneadoug
GitHub user corneadoug opened a pull request: https://github.com/apache/incubator-zeppelin/pull/623 Pass defaultValue parameter to z.select ### What is this PR for? This PR is to fix the `z.select()` defaultValue ### What type of PR is it? Bug Fix ### Is the

[jira] [Created] (ZEPPELIN-595) GUI doesn't display decimal values correctly

2016-01-12 Thread Zhong Wang (JIRA)
Zhong Wang created ZEPPELIN-595: --- Summary: GUI doesn't display decimal values correctly Key: ZEPPELIN-595 URL: https://issues.apache.org/jira/browse/ZEPPELIN-595 Project: Zeppelin Issue Type: B

[jira] [Created] (ZEPPELIN-596) Always use default marshaller in Ignite interpreter

2016-01-12 Thread Andrey Gura (JIRA)
Andrey Gura created ZEPPELIN-596: Summary: Always use default marshaller in Ignite interpreter Key: ZEPPELIN-596 URL: https://issues.apache.org/jira/browse/ZEPPELIN-596 Project: Zeppelin Issu

[GitHub] incubator-zeppelin pull request: ZEPPELIN-596 Use default marshall...

2016-01-12 Thread agura
GitHub user agura opened a pull request: https://github.com/apache/incubator-zeppelin/pull/624 ZEPPELIN-596 Use default marshaller in Ignite interpreter by default You can merge this pull request into a Git repository by running: $ git pull https://github.com/agura/incubator-z

[GitHub] incubator-zeppelin pull request: Tested with local LDAP configurat...

2016-01-12 Thread rconline
GitHub user rconline opened a pull request: https://github.com/apache/incubator-zeppelin/pull/625 Tested with local LDAP configurations. ### What is this PR for? @hayssams @Leemoonsoo Just tested https://github.com/apache/incubator-zeppelin/pull/586 with a local LDAP, works wel

[GitHub] incubator-zeppelin pull request: Utilities for Angular Display sys...

2016-01-12 Thread doanduyhai
Github user doanduyhai commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/591#issuecomment-170899805 @Leemoonsoo Can you provide a screen capture/code sample to show how we can use those utilities function to update an Angular object server-side from the

[GitHub] incubator-zeppelin pull request: Tested with local LDAP configurat...

2016-01-12 Thread hayssams
Github user hayssams commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/625#issuecomment-170908172 @rconline You should probably leave "authcBasic" commented and "anon" uncommented. --- If your project is set up for it, you can reply to this email an

[GitHub] incubator-zeppelin pull request: Tested with local LDAP configurat...

2016-01-12 Thread rconline
Github user rconline commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/625#issuecomment-170909250 @hayssams thanks for pointing out. Made the change. --- 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: Rest Apis to export/import

2016-01-12 Thread swakrish
Github user swakrish commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/614#issuecomment-170977199 @bzz thanks for your comments, i just wanted to confirm since when i run test suite it doesnt break any of my changes :) I will take a look at those

[GitHub] incubator-zeppelin pull request: ZEPPELIN-558 ] that is missing on...

2016-01-12 Thread cloverhearts
Github user cloverhearts closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/621 --- 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 t

[GitHub] incubator-zeppelin pull request: ZEPPELIN-558 ] that is missing on...

2016-01-12 Thread cloverhearts
GitHub user cloverhearts reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/621 ZEPPELIN-558 ] that is missing on geode travis build test. The newly created PR for the previous pr. https://github.com/apache/incubator-zeppelin/pull/599 ### What is

[GitHub] incubator-zeppelin pull request: Zeppelin-540: [WIP] Notebook vers...

2016-01-12 Thread maaark11
Github user maaark11 commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/577#issuecomment-171008532 guys you have some communication channel, I enjoyed the project and want to help --- If your project is set up for it, you can reply to this email and have

[GitHub] incubator-zeppelin pull request: ZEPPELIN-596 Use default marshall...

2016-01-12 Thread agura
Github user agura closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/624 --- 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] incubator-zeppelin pull request: ZEPPELIN-596 Use default marshall...

2016-01-12 Thread agura
GitHub user agura reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/624 ZEPPELIN-596 Use default marshaller in Ignite interpreter by default You can merge this pull request into a Git repository by running: $ git pull https://github.com/agura/incubator

[GitHub] incubator-zeppelin pull request: [WIP] Improve travis build script

2016-01-12 Thread Leemoonsoo
GitHub user Leemoonsoo opened a pull request: https://github.com/apache/incubator-zeppelin/pull/626 [WIP] Improve travis build script ### What is this PR for? Recent Travis CI build is failing with ``` The log length has exceeded the limit of 4 Megabytes (this usual

[GitHub] incubator-zeppelin pull request: [WIP] Improve travis build script

2016-01-12 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/626#issuecomment-171028172 Also how about; env: global: - TERM=dumb read that on stackoverflow, but couldn't find much doc related to it.

[GitHub] incubator-zeppelin pull request: ZEPPELIN-596 Use default marshall...

2016-01-12 Thread c0s
Github user c0s commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/624#issuecomment-171051605 Thanks, this is a way cleaner fix that the one I did in ZEPPELIN-545 - these Spring configs are in my blind spot, unfortunately. +1 Looking through buil

[GitHub] incubator-zeppelin pull request: Rest Apis to export/import

2016-01-12 Thread swakrish
Github user swakrish commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/614#issuecomment-171086165 @bzz , point 3 is negative test case, so i think its fine, the unit test dont fail here --- If your project is set up for it, you can reply to this email a

[GitHub] incubator-zeppelin pull request: Remove paragraph title capitaliza...

2016-01-12 Thread minahlee
GitHub user minahlee opened a pull request: https://github.com/apache/incubator-zeppelin/pull/627 Remove paragraph title capitalization style from css ### What is this PR for? Remove paragraph title font capitalization style from css. ### What type of PR is it? Bug F

[GitHub] incubator-zeppelin pull request: Improve travis build script

2016-01-12 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/626#issuecomment-171106086 Now .travis.yml creates each build job for each build profile instead of run every build profile in a single job. So output message is splitted into diffe

[GitHub] incubator-zeppelin pull request: Improve travis build script

2016-01-12 Thread minahlee
Github user minahlee commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/626#issuecomment-171106177 +1 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 h

[GitHub] incubator-zeppelin pull request: Improve travis build script

2016-01-12 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/626#issuecomment-171106318 While every CI test is failing, i'm merging this patch as a hotfix. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] incubator-zeppelin pull request: Improve travis build script

2016-01-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/626 --- 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: Improve travis build script

2016-01-12 Thread elbamos
Github user elbamos commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/626#issuecomment-171106557 +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 not have this feature

[GitHub] incubator-zeppelin pull request: Remove paragraph title capitaliza...

2016-01-12 Thread minahlee
GitHub user minahlee reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/627 Remove paragraph title capitalization style from css ### What is this PR for? Remove paragraph title font capitalization style from css. ### What type of PR is it? Bug

[GitHub] incubator-zeppelin pull request: Remove paragraph title capitaliza...

2016-01-12 Thread minahlee
Github user minahlee closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/627 --- 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-546 Enables interpreter ...

2016-01-12 Thread minahlee
Github user minahlee commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/590#issuecomment-17761 Rebased master branch and CI 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. If

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-575] Cassandra interpre...

2016-01-12 Thread elbamos
Github user elbamos commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/604#issuecomment-171113980 @doanduyhai Its fine. Its not confusing at all. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as w

[GitHub] incubator-zeppelin pull request: Remove paragraph title capitaliza...

2016-01-12 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/627#issuecomment-171118368 looks 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

[GitHub] incubator-zeppelin pull request: Remove paragraph title capitaliza...

2016-01-12 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/627#issuecomment-171118863 Should we also do it for Notebook Title? --- 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: Pass defaultValue parameter to z....

2016-01-12 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/623#issuecomment-171120829 Triggering CI again --- 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 n

[GitHub] incubator-zeppelin pull request: Pass defaultValue parameter to z....

2016-01-12 Thread corneadoug
Github user corneadoug closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/623 --- 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: Pass defaultValue parameter to z....

2016-01-12 Thread corneadoug
GitHub user corneadoug reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/623 Pass defaultValue parameter to z.select ### What is this PR for? This PR is to fix the `z.select()` defaultValue ### What type of PR is it? Bug Fix ### Is t

[GitHub] incubator-zeppelin pull request: Remove paragraph title capitaliza...

2016-01-12 Thread minahlee
Github user minahlee commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/627#issuecomment-171123889 @corneadoug It is already possible to have lowercase character for the beggining of the word in notebook title --- If your project is set up for it, you ca

[jira] [Created] (ZEPPELIN-597) Change Zeppelin REST API Error Code

2016-01-12 Thread CORNEAU Damien (JIRA)
CORNEAU Damien created ZEPPELIN-597: --- Summary: Change Zeppelin REST API Error Code Key: ZEPPELIN-597 URL: https://issues.apache.org/jira/browse/ZEPPELIN-597 Project: Zeppelin Issue Type: Im

[GitHub] incubator-zeppelin pull request: Remove paragraph title capitaliza...

2016-01-12 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/627#issuecomment-171125070 Awesome then +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 not h

[GitHub] incubator-zeppelin pull request: Pass defaultValue parameter to z....

2016-01-12 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/623#issuecomment-171128177 READY! --- 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

[GitHub] incubator-zeppelin pull request: ZEPPELIN-546 Enables interpreter ...

2016-01-12 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/590#issuecomment-171128338 Great. 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: Tested with local LDAP configurat...

2016-01-12 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/625#issuecomment-171136129 @rconline Thanks a lot for sharing this documentation. CI build failing is irrelevant. LGTM --- If your project is set up for it, you can reply to t

[GitHub] incubator-zeppelin pull request: Remove paragraph title capitaliza...

2016-01-12 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/627#issuecomment-171140148 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

Re: [DISCUSSION] new page which shows variables for Zeppelin

2016-01-12 Thread Jungtaek Lim
Jongyoul, Yes, hiding password only is a great start for me. We can review and discuss what to remove on actual PR later. I don't have machine for develop for now (it may take some days to get), so I'll get into it when I'm available. Thanks for the kind opinion! Jungtaek Lim (HeartSaVioR) On

[GitHub] incubator-zeppelin pull request: Restore Navbar connection status ...

2016-01-12 Thread sejunra
GitHub user sejunra opened a pull request: https://github.com/apache/incubator-zeppelin/pull/628 Restore Navbar connection status for websocket only ### What is this PR for? After the Shiro PR was merged, the Navbar status was changed to include the token status/name, however th

[GitHub] incubator-zeppelin pull request: ZEPPELIN-546 Enables interpreter ...

2016-01-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/590 --- 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

[jira] [Created] (ZEPPELIN-598) Dynamic loading for Interpreter jar

2016-01-12 Thread Chae-Sung Lim (JIRA)
Chae-Sung Lim created ZEPPELIN-598: -- Summary: Dynamic loading for Interpreter jar Key: ZEPPELIN-598 URL: https://issues.apache.org/jira/browse/ZEPPELIN-598 Project: Zeppelin Issue Type: Imp

[GitHub] incubator-zeppelin pull request: Restore Navbar connection status ...

2016-01-12 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/628#issuecomment-171150156 Thanks, Tested, LGTM The `authenticated` status (username) could probably be shown in the navbar in the future if we have some other actions like logou

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-551] Add paragraph scop...

2016-01-12 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/588#issuecomment-171154324 ready to merge --- 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-551] Add paragraph scop...

2016-01-12 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/588#issuecomment-171156481 I will take a look at it today then :) --- 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-551] Add paragraph scop...

2016-01-12 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/588#issuecomment-171161239 Do you have some example or documentation on how you can bind at paragraph or notebook level from UI? --- If your project is set up for it, you can reply

[GitHub] incubator-zeppelin pull request: Remove paragraph title capitaliza...

2016-01-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/627 --- 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-598 ] Dynamic loading fo...

2016-01-12 Thread cloverhearts
GitHub user cloverhearts opened a pull request: https://github.com/apache/incubator-zeppelin/pull/629 ZEPPELIN-598 ] Dynamic loading for Interpreter ### What is this PR for? Use of external libraries that are included in the Roadmap, The interpreter needs related to dynamic l

[GitHub] incubator-zeppelin pull request: ZEPPELIN-598 ] Dynamic loading fo...

2016-01-12 Thread cloverhearts
Github user cloverhearts closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/629 --- 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 t

[jira] [Created] (ZEPPELIN-599) notebook search should search paragraph title

2016-01-12 Thread Felix Cheung (JIRA)
Felix Cheung created ZEPPELIN-599: - Summary: notebook search should search paragraph title Key: ZEPPELIN-599 URL: https://issues.apache.org/jira/browse/ZEPPELIN-599 Project: Zeppelin Issue Ty

[jira] [Created] (ZEPPELIN-600) notebook search should have a way to clear search and return to the previous page

2016-01-12 Thread Felix Cheung (JIRA)
Felix Cheung created ZEPPELIN-600: - Summary: notebook search should have a way to clear search and return to the previous page Key: ZEPPELIN-600 URL: https://issues.apache.org/jira/browse/ZEPPELIN-600

Re: [VOTE] Release Apache Zeppelin (incubating) 0.5.6-incubating (RC2)

2016-01-12 Thread Alexander Bezzubov
+1 (binding) On Tue, Jan 12, 2016 at 9:55 AM, Jongyoul Lee wrote: > +1 (binding), looks good to move forward to Spark 1.6 > > On Mon, Jan 11, 2016 at 11:27 PM, Alexander Bezzubov > > wrote: > > > Hi, > > > > I am pleased to be calling this vote for the following RC2 to be release > as > > Apach

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-551] Add paragraph scop...

2016-01-12 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/588#issuecomment-171168129 Let me update javadoc --- 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-601) import note, add from url textbox is not clearing buttons from previous choices

2016-01-12 Thread Felix Cheung (JIRA)
Felix Cheung created ZEPPELIN-601: - Summary: import note, add from url textbox is not clearing buttons from previous choices Key: ZEPPELIN-601 URL: https://issues.apache.org/jira/browse/ZEPPELIN-601 P

[GitHub] incubator-zeppelin pull request: Pass defaultValue parameter to z....

2016-01-12 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/623#issuecomment-171168297 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

[jira] [Created] (ZEPPELIN-602) elasticsearch throws ArrayIndexOutOfBoundsException for interpreting an empty paragraph

2016-01-12 Thread Felix Cheung (JIRA)
Felix Cheung created ZEPPELIN-602: - Summary: elasticsearch throws ArrayIndexOutOfBoundsException for interpreting an empty paragraph Key: ZEPPELIN-602 URL: https://issues.apache.org/jira/browse/ZEPPELIN-602

RE: [VOTE] Release Apache Zeppelin (incubating) 0.5.6-incubating (RC2)

2016-01-12 Thread Felix Cheung
+1 Tested: packaged spark, external sparkSpark/Scala, pyspark, md, sh, flink, various UI features Found and filed these bugs, I don't think they are blockers: https://issues.apache.org/jira/browse/ZEPPELIN-599 notebook search should search paragraph titlehttps://issues.apache.org/jira/browse

[GitHub] incubator-zeppelin pull request: Rest Apis to export/import

2016-01-12 Thread bzz
Github user bzz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/614#issuecomment-171173477 Great! One more thing - @swakrish @corneadoug could you guys please point me to the place where REST API and WS API for importing\exporting notebooks re-us

[GitHub] incubator-zeppelin pull request: ZEPPELIN-598 ] Dynamic loading fo...

2016-01-12 Thread cloverhearts
GitHub user cloverhearts opened a pull request: https://github.com/apache/incubator-zeppelin/pull/630 ZEPPELIN-598 ] Dynamic loading for Interpreter ### What is this PR for? Use of external libraries that are included in the Roadmap, The interpreter needs related to dynamic l

[GitHub] incubator-zeppelin pull request: ZEPPELIN-598 ] Dynamic loading fo...

2016-01-12 Thread cloverhearts
Github user cloverhearts closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/630 --- 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 t

[GitHub] incubator-zeppelin pull request: Rest Apis to export/import

2016-01-12 Thread swakrish
Github user swakrish commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/614#issuecomment-171176416 @bzz Please take a look at notebook.java, this is accessible both from REST and Websocket, We need to modify notebookserver.java to use the common method

[GitHub] incubator-zeppelin pull request: Utilities for Angular Display sys...

2016-01-12 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/591#issuecomment-171176477 @doanduyhai The examples in this PR's description can be run on %spark interpreter. Currently, push new angular object from front-end is not possible.

[GitHub] incubator-zeppelin pull request: Utilities for Angular Display sys...

2016-01-12 Thread doanduyhai
Github user doanduyhai commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/591#issuecomment-171177451 Thanks for the clarifications @Leemoonsoo. I'll try to push a small PR for the front-end angular value push --- If your project is set up for it, you can

[GitHub] incubator-zeppelin pull request: ZEPPELIN-598 ] Dynamic loading fo...

2016-01-12 Thread cloverhearts
GitHub user cloverhearts opened a pull request: https://github.com/apache/incubator-zeppelin/pull/631 ZEPPELIN-598 ] Dynamic loading for Interpreter ### What is this PR for? Use of external libraries that are included in the Roadmap, The interpreter needs related to dynamic l

[GitHub] incubator-zeppelin pull request: ZEPPELIN-596 Use default marshall...

2016-01-12 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/624#issuecomment-171178149 CI test failure is irrelevant. 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 yo

[GitHub] incubator-zeppelin pull request: ZEPPELIN-596 Use default marshall...

2016-01-12 Thread c0s
Github user c0s commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/624#issuecomment-171179270 Moon, could you merge it then? Looks like I don't have the permissions. --- 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 546](documentation) Ena...

2016-01-12 Thread AhyoungRyu
Github user AhyoungRyu commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/609#issuecomment-171179305 @corneadoug Thank you for your prudent review. I applied all of your comment in this docs : ) --- If your project is set up for it, you can reply to this

[GitHub] incubator-zeppelin pull request: ZEPPELIN-596 Use default marshall...

2016-01-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/624 --- 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: Restore Navbar connection status ...

2016-01-12 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/628#issuecomment-171180273 LGTM. Thanks for the contribution. --- 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-598 ] Dynamic loading fo...

2016-01-12 Thread bzz
Github user bzz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/631#issuecomment-171181098 Looks good to me, except for major style\formatting nit above. --- If your project is set up for it, you can reply to this email and have your reply appear on Gi

[GitHub] incubator-zeppelin pull request: ZEPPELIN-598 ] Dynamic loading fo...

2016-01-12 Thread cloverhearts
Github user cloverhearts commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/631#issuecomment-171182487 @bzz Thank you for feedback! I will modify as you advise. --- If your project is set up for it, you can reply to this email and have your reply appe

[GitHub] incubator-zeppelin pull request: Fix some typos, grammars and Incr...

2016-01-12 Thread AhyoungRyu
Github user AhyoungRyu commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/578#issuecomment-171183165 @corneadoug @felixcheung I applied the comments : ) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[jira] [Created] (ZEPPELIN-603) Modify UI notebook import to use common backend code

2016-01-12 Thread Ramaswamy Devarajan (JIRA)
Ramaswamy Devarajan created ZEPPELIN-603: Summary: Modify UI notebook import to use common backend code Key: ZEPPELIN-603 URL: https://issues.apache.org/jira/browse/ZEPPELIN-603 Project: Zeppel

[GitHub] incubator-zeppelin pull request: Fix some typos, grammars and Incr...

2016-01-12 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/578#issuecomment-171198503 Honestly, there is too much changes in that PR to properly review all the grammar and sentences. But I think that overall the changes are good, and peo

[GitHub] incubator-zeppelin pull request: [Zeppelin 546](documentation) Ena...

2016-01-12 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/609#issuecomment-171200014 Thanks, We can merge it when the other PR is merges :) --- If your project is set up for it, you can reply to this email and have your reply appear on Git

[GitHub] incubator-zeppelin pull request: Tested with local LDAP configurat...

2016-01-12 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/625#issuecomment-171208925 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