[GitHub] incubator-zeppelin pull request: ZEPPELIN-406 Handle license issue...

2015-11-10 Thread khalidhuseynov
Github user khalidhuseynov commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/407#issuecomment-155679417 +1 for 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

[GitHub] incubator-zeppelin pull request: ZEPPELIN-406 Handle license issue...

2015-11-10 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/407#issuecomment-155678369 I'll merge if there're no more discussions and restart vote with rc3 --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] incubator-zeppelin pull request: ZEPPELIN-293 notebook execution r...

2015-11-10 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/293#issuecomment-155670616 I tried it again, this still works here is what i do to reproduce (without fix) https://issues.apache.org/jira/secure/attachment/12771695/easy%20st

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-407] Improve document o...

2015-11-10 Thread minahlee
Github user minahlee commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/410#issuecomment-155661742 @felixcheung I'd like to help [ZEPPELIN-368](https://issues.apache.org/jira/browse/ZEPPELIN-368?jql=project%20%3D%20ZEPPELIN). Do you mind if I create anoth

[jira] [Created] (ZEPPELIN-411) JDK8 support

2015-11-10 Thread Lee moon soo (JIRA)
Lee moon soo created ZEPPELIN-411: - Summary: JDK8 support Key: ZEPPELIN-411 URL: https://issues.apache.org/jira/browse/ZEPPELIN-411 Project: Zeppelin Issue Type: Improvement Repor

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-328] Interpreter page s...

2015-11-10 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/411#issuecomment-155652264 looks good. agreed multi-binding of the interpreter is confusing. also we might need to support multiple working instances too. --- If your project

[GitHub] incubator-zeppelin pull request: Zeppelin jvm mem options specific...

2015-11-10 Thread piyush-mukati
Github user piyush-mukati closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/313 --- 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] incubator-zeppelin pull request: ZEPPELIN-387 Allow enable/disable...

2015-11-10 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/414#issuecomment-155631510 +1 for Enable/Disable run than Enable/Disable paragraph. --- If your project is set up for it, you can reply to this email and have your reply appear on G

[GitHub] incubator-zeppelin pull request: ZEPPELIN-265 auto save on page na...

2015-11-10 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/357 --- 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-410 Autoscroll only in y...

2015-11-10 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/419#issuecomment-155629746 Create output larger than paragraph width, eg ``` println("""%html output""") ``` and create other paragraphs a head of it, long e

[GitHub] incubator-zeppelin pull request: Update documentation: Set informa...

2015-11-10 Thread anthonycorbacho
Github user anthonycorbacho closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/265 --- 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 i

[GitHub] incubator-zeppelin pull request: ZEPPELIN-410 Autoscroll only in y...

2015-11-10 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/419#issuecomment-155625519 Any easy way to reproduce this, for me to test this PR? --- 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-387 Allow enable/disable...

2015-11-10 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/414#issuecomment-155625281 After thinking about it more, _Enable paragraph_ and _Disable paragraph_ can be a bit confusing since the paragraph is still shown. _Enable run_ a

[GitHub] incubator-zeppelin pull request: ZEPPELIN-383 Override classpath w...

2015-11-10 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/412#issuecomment-155619296 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: Zeppelin Flink Spark tutorial

2015-11-10 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/418#issuecomment-155618776 I see, so this is referring to 0.10 Flink only? " http://trevorgrant.org/2015/11/03/apache-casserole-a-delicious-big-data-recipe-for-the-whole-fam

[GitHub] incubator-zeppelin pull request: ZEPPELIN-383 Override classpath w...

2015-11-10 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/412#issuecomment-155613262 merge 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 GitHub as well. If y

[GitHub] incubator-zeppelin pull request: Zeppelin Flink Spark tutorial

2015-11-10 Thread rawkintrevo
Github user rawkintrevo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/418#issuecomment-155604728 No. Just the Flink 0.9 release that Zeppelin runs on internally. --- If your project is set up for it, you can reply to this email and have your reply a

[GitHub] incubator-zeppelin pull request: Corrected and updated the MapR Di...

2015-11-10 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/391#issuecomment-155602970 Thanks for taking care profiles for MapR distribution. Looks good to me and merge if there're no more discussions. --- If your project is set up for i

Re: [GitHub] incubator-zeppelin pull request: Corrected and updated the MapR Di...

2015-11-10 Thread Paul Curtis
Done and pushed. Regards. On Tue, Nov 10, 2015 at 6:15 PM, Leemoonsoo wrote: > Github user Leemoonsoo commented on the pull request: > > > https://github.com/apache/incubator-zeppelin/pull/391#issuecomment-155598530 > > Thanks, Could you also handle repository section in hive/pom.xml? > > >

Re: [GitHub] incubator-zeppelin pull request: Corrected and updated the MapR Di...

2015-11-10 Thread Paul Curtis
As the Hive POM doesn't use profiles, and there isn't a direct relationship between MapR versions and Hive (ie. you can install different version of Hive on the same version of MapR), I will comment out the MapR repository. This will be added to any release notes MapR provides to our customers usin

[GitHub] incubator-zeppelin pull request: ZEPPELIN-401 Improve autoscroll

2015-11-10 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/400#issuecomment-155601993 @gauravkumar37 Thanks for sharing a problem. I have created a fix https://github.com/apache/incubator-zeppelin/pull/419 --- If your project is set up for

[GitHub] incubator-zeppelin pull request: ZEPPELIN-410 Autoscroll only in y...

2015-11-10 Thread Leemoonsoo
GitHub user Leemoonsoo opened a pull request: https://github.com/apache/incubator-zeppelin/pull/419 ZEPPELIN-410 Autoscroll only in y axis Addresses https://issues.apache.org/jira/browse/ZEPPELIN-410 Autoscroll scrolls not only vertical, but also horizontal scrollbar. but horizo

[jira] [Created] (ZEPPELIN-410) Autoscroll should not do horizontal scroll

2015-11-10 Thread Lee moon soo (JIRA)
Lee moon soo created ZEPPELIN-410: - Summary: Autoscroll should not do horizontal scroll Key: ZEPPELIN-410 URL: https://issues.apache.org/jira/browse/ZEPPELIN-410 Project: Zeppelin Issue Type:

[GitHub] incubator-zeppelin pull request: Corrected and updated the MapR Di...

2015-11-10 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/391#issuecomment-155598530 Thanks, Could you also handle repository section in hive/pom.xml? --- If your project is set up for it, you can reply to this email and have your reply ap

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-407] Improve document o...

2015-11-10 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/410#issuecomment-155596456 looks good, this covers most of this earlier bug: ZEPPELIN-368 I'd say we would need to add: complete list of properties supported by interpreters (eg

[GitHub] incubator-zeppelin pull request: [WIP] ZEPPELIN-52 Long statements...

2015-11-10 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/415#issuecomment-155594838 Here's little bit of history. Currently, Zeppelin has 3 Looknfeels. 'default', 'simple' and 'report'. But before the looknfeel feature, there were

[GitHub] incubator-zeppelin pull request: Zeppelin Flink Spark tutorial

2015-11-10 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/418#issuecomment-155594138 If I understand correctly this requires a special build of Flink? --- If your project is set up for it, you can reply to this email and have your reply a

[GitHub] incubator-zeppelin pull request: Zeppelin Flink Spark tutorial

2015-11-10 Thread rawkintrevo
GitHub user rawkintrevo opened a pull request: https://github.com/apache/incubator-zeppelin/pull/418 Zeppelin Flink Spark tutorial Moon recommended I migrate a How To tutorial I wrote into the website. http://mail-archives.apache.org/mod_mbox/incubator-zeppelin-users/20151

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-409] upgrade spark to 1...

2015-11-10 Thread galleon
GitHub user galleon opened a pull request: https://github.com/apache/incubator-zeppelin/pull/417 [ZEPPELIN-409] upgrade spark to 1.5.2 Upgrade spark to latest (1.5.2) version You can merge this pull request into a Git repository by running: $ git pull https://github.com/galleon

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-409] Upgrade spark to 1...

2015-11-10 Thread galleon
Github user galleon closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/416 --- 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 fe

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-409] Upgrade spark to 1...

2015-11-10 Thread galleon
GitHub user galleon opened a pull request: https://github.com/apache/incubator-zeppelin/pull/416 [ZEPPELIN-409] Upgrade spark to 1.5.2 Upgrade spark to latest 1.5.2 version. Changed files: - spark-dependencies/pom.xml - spark/src/main/java/org/apache/zeppelin/spark/Sp

[jira] [Created] (ZEPPELIN-409) Upgrade to spark 1.5.2

2015-11-10 Thread Guillaume Alleon (JIRA)
Guillaume Alleon created ZEPPELIN-409: - Summary: Upgrade to spark 1.5.2 Key: ZEPPELIN-409 URL: https://issues.apache.org/jira/browse/ZEPPELIN-409 Project: Zeppelin Issue Type: Improvement

[GitHub] incubator-zeppelin pull request: Corrected and updated the MapR Di...

2015-11-10 Thread pfcurtis
Github user pfcurtis commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/391#issuecomment-155489946 I have updated the Spark POM (spark-dependencies) to move the MapR repositories to the individual profiles. I have pushed the changes to my fork. Re

[GitHub] incubator-zeppelin pull request: ZEPPELIN-401 Improve autoscroll

2015-11-10 Thread gauravkumar37
Github user gauravkumar37 commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/400#issuecomment-155478665 Thanks for the bug fix. I noticed that it works great when there is no horizontal scrolling. However, when there is output of paragraphs which exten

[GitHub] incubator-zeppelin pull request: ZEPPELIN-315 Fix time-dependant s...

2015-11-10 Thread FRosner
Github user FRosner commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/315#issuecomment-155467136 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 featu

[GitHub] incubator-zeppelin pull request: [WIP] ZEPPELIN-52 Long statements...

2015-11-10 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/415#issuecomment-155457111 @Leemoonsoo how about we hide the control bar and show it on mouseover event of the paragraph --- If your project is set up for it, you can reply to this e

[GitHub] incubator-zeppelin pull request: ZEPPELIN-315 Fix time-dependant s...

2015-11-10 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/315#issuecomment-155444690 Okay, Merging it 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 GitHub a

[GitHub] incubator-zeppelin pull request: ZEPPELIN-406 Handle license issue...

2015-11-10 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/407#issuecomment-155437858 3rd party dependency licenses are handled /LICENSE - for source package /NOTICE - for source package /licenses - copies of 3rd party depe

[GitHub] incubator-zeppelin pull request: ZEPPELIN-52 Long statements in pa...

2015-11-10 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/415#issuecomment-155420038 @Leemoonsoo Thanks for the document. >...without adding more empty spaces inside of Paragraph... makes sense. working on a better solution! ---

[GitHub] incubator-zeppelin pull request: ZEPPELIN-401 Improve autoscroll

2015-11-10 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/400 --- 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: Corrected and updated the MapR Di...

2015-11-10 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/391#issuecomment-155410179 According to https://issues.apache.org/jira/browse/ZEPPELIN-404, MapR maven repository shouldn't be activated by default. Could you move MapR repository i

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-394] Update cassandra-s...

2015-11-10 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/395 --- 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-52 Long statements in pa...

2015-11-10 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/415#issuecomment-155399747 Thanks for the improvement. There was design principles when Zeppelin UI implemented in the beginning. I just documented that https://cwiki.apache.

[GitHub] incubator-zeppelin pull request: ZEPPELIN-385 Read-only mode for z...

2015-11-10 Thread Xrixcis
Github user Xrixcis commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/389#issuecomment-155395535 Hi, great feature, thanks! One little bug I came across, isReadOnly function seems to be missing in paragraph.controller.js. --- If your project is set

[GitHub] incubator-zeppelin pull request: ZEPPELIN-52 Long statements in pa...

2015-11-10 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/415#issuecomment-155385350 tested, and this works for me. --- 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-387 Allow enable/disable...

2015-11-10 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/414#issuecomment-155385289 Hi @corneadoug, That sounds like a right spot for the enable/disable menu item. Updated in 87aa4e8. Please take a look at the new screenshots. Thanks!

[GitHub] incubator-zeppelin pull request: ZEPPELIN-52 Long statements in pa...

2015-11-10 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/415#issuecomment-155382830 @prabhjyotsingh tested on Firefox and IE :tada: --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as we

[GitHub] incubator-zeppelin pull request: ZEPPELIN-265 auto save on page na...

2015-11-10 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/357#issuecomment-155374528 Confirmed on: Firefox, Safari, Opera, Chrome, IE10 Good to go! I will merge tomorrow if there is no more discussions --- If your project is s

[GitHub] incubator-zeppelin pull request: ZEPPELIN-265 auto save on page na...

2015-11-10 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/357#issuecomment-155364453 @corneadoug thanks, same thing was happening with Firefox and edge, this should take care of both and rest (tested on IE10, FF, Chrome, Edge, Safari)

[GitHub] incubator-zeppelin pull request: ZEPPELIN-265 auto save on page na...

2015-11-10 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/357#issuecomment-155364096 Firefox is acting pretty strange, I just realized that Firefox and Safari are not loading some HTML and Markdown results when loading a notebook...

[GitHub] incubator-zeppelin pull request: ZEPPELIN-265 auto save on page na...

2015-11-10 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/357#issuecomment-155360146 yeah, I tried same think with jQuery as well, wasn't working, so thought its something with my firefox. --- If your project is set up for it, you can

[GitHub] incubator-zeppelin pull request: ZEPPELIN-265 auto save on page na...

2015-11-10 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/357#issuecomment-155358771 Fun thing is the save on unload is not working on Firefox. Everything is sent correctly, But the server never receive the Websocket message --- If yo

[GitHub] incubator-zeppelin pull request: ZEPPELIN-315 Fix time-dependant s...

2015-11-10 Thread RPCMoritz
Github user RPCMoritz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/315#issuecomment-155355447 @Leemoonsoo I'd really like to be able to tick this one off of my list - any chance of merging this soon? --- If your project is set up for it, you can re