[GitHub] zeppelin issue #1010: [ZEPPELIN-998] Extend install.md -> Quick Start

2016-06-15 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1010 @AhyoungRyu great job! Let me take another pass on it plz. BTW is removal of `conf/shiro.ini` a part of the quickstart? --- If your project is set up for it, you can reply to this email and h

Re: Problem on importing Notebook

2016-06-15 Thread Andrea Santurbano
Hi thanks for the response! When this will be merged? This an important feature... Il giorno mer 15 giu 2016 alle ore 06:06 Hyung Sung Shim ha scritto: > Hi. > It's a bug and is fixing in [1] PR. > > [1] > https://github.com/apache/zeppelin/pull/933 > > 2016-06-15 1:39 GMT+09:00 Andrea Santurban

Re: Problem on importing Notebook

2016-06-15 Thread Hyung Sung Shim
Hi. You are definitely right! I believe it'll be merged soon. 2016-06-15 16:02 GMT+09:00 Andrea Santurbano : > Hi thanks for the response! > When this will be merged? > This an important feature... > > Il giorno mer 15 giu 2016 alle ore 06:06 Hyung Sung Shim < > hss...@nflabs.com> > ha scritto:

[GitHub] zeppelin issue #1010: [ZEPPELIN-998] Extend install.md -> Quick Start

2016-06-15 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1010 @bzz Oh my. That's my bad. I'll revert it now :) --- 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] zeppelin issue #1010: [ZEPPELIN-998] Extend install.md -> Quick Start

2016-06-15 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1010 Thanks for prompt response! Your documentation update looks great to me, modulo few minor suggestions above. Please feel free to use them at your will, otherwise I think it's ready to be

[GitHub] zeppelin pull request #1015: [ZEPPPELIN-1009] Trying to Fix selenium modal e...

2016-06-15 Thread corneadoug
GitHub user corneadoug opened a pull request: https://github.com/apache/zeppelin/pull/1015 [ZEPPPELIN-1009] Trying to Fix selenium modal error ### What is this PR for? This is to check Selenium test failing on Master and #979 ### What type of PR is it? Flacky te

[GitHub] zeppelin issue #1010: [ZEPPELIN-998] Extend install.md -> Quick Start

2016-06-15 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1010 @bzz Thanks for all of your suggestions again. I've just addressed them. Please check my last commit :) --- If your project is set up for it, you can reply to this email and have your reply ap

[GitHub] zeppelin issue #933: [ZEPPELIN-905] fix failed notebook import bug

2016-06-15 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/933 @swakrish do you think it's ready to be merged? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this featur

[GitHub] zeppelin issue #993: [ZEPPELIN-987] Enable user to secure interpreter settin...

2016-06-15 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/993 @prabhjyotsingh Yeah it makes sense. It would be better. Let me figure out it then :) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as w

[GitHub] zeppelin pull request #1016: Bump up version to 0.7.0-SNAPSHOT

2016-06-15 Thread minahlee
GitHub user minahlee opened a pull request: https://github.com/apache/zeppelin/pull/1016 Bump up version to 0.7.0-SNAPSHOT ### What is this PR for? Bump up version to 0.7.0-SNAPSHOT You can merge this pull request into a Git repository by running: $ git pull https://github

[GitHub] zeppelin issue #989: Added ipfsnotebookrepo [ Zeppelin-683 WIP ]

2016-06-15 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/989 @onkarshedge I understand that it's WIP, just wanted to remind that eventually your `NotebookRepo` implementation should user `NotebookRepoVersioned`, see Git one as an example. Could you also

[GitHub] zeppelin pull request #1000: typo

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

[GitHub] zeppelin issue #994: [ZEPPELIN-986] Create publish release script

2016-06-15 Thread echarles
Github user echarles commented on the issue: https://github.com/apache/zeppelin/pull/994 why spark is not published. is it related to the assembly? Btw, spark 2 doesn't use assembly any more and favor dependencies in separate jars. See https://issues.apache.org/jira/browse/SPA

[GitHub] zeppelin issue #994: [ZEPPELIN-986] Create publish release script

2016-06-15 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/994 Great, I think if checking error conditions could be added to network-related commands - it would be even more awesome, as this is quite common failure pattern. Other then that, looks great to

[GitHub] zeppelin issue #994: [ZEPPELIN-986] Create publish release script

2016-06-15 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/994 @echarles I think my description caused confusion. Spark will be included in release since most of Zeppelin user is based on Spark. The modules I wrote in description is skipped modules for publis

[GitHub] zeppelin issue #994: [ZEPPELIN-986] Create publish release script

2016-06-15 Thread echarles
Github user echarles commented on the issue: https://github.com/apache/zeppelin/pull/994 Didn't look at the code... just saw in the description (and @minahlee just confirmed it would be published). Thinking more about this PR: - Why the deploy and release plugins were no

[GitHub] zeppelin issue #976: ZEPPLIN-976 ] HotFix - zeppelin server does not restart...

2016-06-15 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/976 @cloverhearts thank you for adding tests! Looks great to me, 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] zeppelin issue #994: [ZEPPELIN-986] Create publish release script

2016-06-15 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/994 Thank you for the feedback @echarles ! I think it should be fine to add missing bits, as soon as they are identified and there is somebody with actual use case for it later on. Would you be wi

[jira] [Created] (ZEPPELIN-1011) Add force layout (graph) visualization in Zeppelin Web notebook

2016-06-15 Thread David Tzoor (JIRA)
David Tzoor created ZEPPELIN-1011: - Summary: Add force layout (graph) visualization in Zeppelin Web notebook Key: ZEPPELIN-1011 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1011 Project: Zeppel

[GitHub] zeppelin issue #994: [ZEPPELIN-986] Create publish release script

2016-06-15 Thread echarles
Github user echarles commented on the issue: https://github.com/apache/zeppelin/pull/994 Good to hear the release plugin is working fine. The release manager is responsible for the tools he wants. Fine to me if it is a script or a plugin, but at the end the artifacts must find

[GitHub] zeppelin pull request #1017: Add force layout visualization in paragraphs fo...

2016-06-15 Thread davidtzoor
GitHub user davidtzoor opened a pull request: https://github.com/apache/zeppelin/pull/1017 Add force layout visualization in paragraphs for SQL queries ### What is this PR for? Add force layout (graph) visualization to the existing set of visualizations in Zeppelin Web notp.

[GitHub] zeppelin pull request #976: ZEPPLIN-976 ] HotFix - zeppelin server does not ...

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

[GitHub] zeppelin issue #1017: Add force layout visualization in paragraphs for SQL q...

2016-06-15 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1017 @davidtzoor great to see new visualization! Could you rebase on top of the latest master please? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] zeppelin issue #1015: [ZEPPPELIN-1009] Trying to Fix selenium modal error

2016-06-15 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1015 Two CI failures here 1. **Python test failure** [logs](https://s3.amazonaws.com/archive.travis-ci.org/jobs/137725472/log.txt) AFAIK never seen before (might be related to #1009)

[GitHub] zeppelin issue #1015: [ZEPPPELIN-1009] Trying to Fix selenium modal error

2016-06-15 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1015 @corneadoug as a first step, how about re-open the PR to trigger CI and see, which ones are reproducible by posting same report for new CI build results? --- If your project is set up for it, you ca

[GitHub] zeppelin issue #1015: [ZEPPPELIN-1009] Trying to Fix selenium modal error

2016-06-15 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1015 Only both errors about delete button are there after running the 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

[GitHub] zeppelin issue #1015: [ZEPPPELIN-1009] Trying to Fix selenium modal error

2016-06-15 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1015 No problem running the test locally --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature en

[ANNOUNCE] branch-0.6 cut

2016-06-15 Thread mina lee
Hi, as we discussed in 0.6.0 release discussion thread [1] I've cut branch-0.6. 300+ patches from 70+ contributors have been merged since last release. Thanks everyone for enriching Zeppelin community. There are still few patches left to be merged before preparing release publish and they are man

[GitHub] zeppelin issue #1005: ZEPPELIN-995 Change scheduler for JDBC interpreter to ...

2016-06-15 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1005 @Leemoonsoo I've totally understood your idea and agree with you. I'll patch for it and push it again --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] zeppelin issue #1017: Add force layout visualization in paragraphs for SQL q...

2016-06-15 Thread davidtzoor
Github user davidtzoor commented on the issue: https://github.com/apache/zeppelin/pull/1017 @bzz Thanks! you mean to rebase the current `apache:master` to my forked repo (`datvidtzoor:master`), right? After I do that, what should I do? --- If your project is set up for it, you can re

[GitHub] zeppelin issue #1012: ZEPPELIN-999 Support alias for JDBC properties

2016-06-15 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1012 @bzz I'll merge it into master and branch-0.6 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this fe

[GitHub] zeppelin pull request #1002: ZEPPELIN-992 Move some tests from InterpreterFa...

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

[GitHub] zeppelin issue #746: [ZEPPELIN-699] Add new synchronous paragraph run REST A...

2016-06-15 Thread doanduyhai
Github user doanduyhai commented on the issue: https://github.com/apache/zeppelin/pull/746 Rebased from master --- 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

[GitHub] zeppelin issue #989: Added ipfsnotebookrepo [ Zeppelin-683 WIP ]

2016-06-15 Thread onkarshedge
Github user onkarshedge commented on the issue: https://github.com/apache/zeppelin/pull/989 @bzz I had a doubt about versioning approach. Currently I am saving(ipfs add 'note') the note in IPFS for every call to save function. This gives me a new multihash on saving. So for version

[GitHub] zeppelin issue #989: Added ipfsnotebookrepo [ Zeppelin-683 WIP ]

2016-06-15 Thread onkarshedge
Github user onkarshedge commented on the issue: https://github.com/apache/zeppelin/pull/989 Should i save to Filesystem like `VFSNotebookRepo` and only when user chooses to version it add it to ipfs that way the ipfs will contain only the ipfs object corresponding to versioned notes a

[GitHub] zeppelin issue #972: ZEPPELIN-963 ] Jobmanagement - (1) basic backend.

2016-06-15 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/972 Merging it into master branch only. --- 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 enabl

[jira] [Created] (ZEPPELIN-1012) Support alias for interpreter

2016-06-15 Thread Jongyoul Lee (JIRA)
Jongyoul Lee created ZEPPELIN-1012: -- Summary: Support alias for interpreter Key: ZEPPELIN-1012 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1012 Project: Zeppelin Issue Type: New Feat

[GitHub] zeppelin pull request #972: ZEPPELIN-963 ] Jobmanagement - (1) basic backend...

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

[GitHub] zeppelin issue #993: [ZEPPELIN-987] Enable user to secure interpreter settin...

2016-06-15 Thread echarles
Github user echarles commented on the issue: https://github.com/apache/zeppelin/pull/993 @AhyoungRyu If you can come to an implementation that fits @prabhjyotsingh good idea, it would be great if you could make it `generic`, with a configurable list of functions that reflect in to the

[GitHub] zeppelin pull request #982: ZEPPELIN-934 Merge Phoenix interpreter into JDBC...

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

[GitHub] zeppelin pull request #982: ZEPPELIN-934 Merge Phoenix interpreter into JDBC...

2016-06-15 Thread jongyoul
GitHub user jongyoul reopened a pull request: https://github.com/apache/zeppelin/pull/982 ZEPPELIN-934 Merge Phoenix interpreter into JDBC interpreter ### What is this PR for? Removing Phoenix interpreter and adding example for using it through JDBC ### What type of PR i

Re: Problem on importing Notebook

2016-06-15 Thread Federico Valeri
Yes, it is planned with the next release 0.6.0 [1] [1] https://issues.apache.org/jira/browse/ZEPPELIN-889 2016-06-15 9:08 GMT+02:00 Hyung Sung Shim : > Hi. > You are definitely right! > I believe it'll be merged soon. > > > 2016-06-15 16:02 GMT+09:00 Andrea Santurbano : > > > Hi thanks for the r

What tool for GIF webapp previews?

2016-06-15 Thread Federico Valeri
Hi all, can I ask what tool do you use to create those useful preview GIFs that we see in Github PRs?

Re: What tool for GIF webapp previews?

2016-06-15 Thread Corneau Damien
I think most of the Mac OSX users, are using licecap. (However beware if using multiple monitor, it only record on the main one) On Wed, Jun 15, 2016 at 9:47 PM, Federico Valeri wrote: > Hi all, can I ask what tool do you use to create those useful preview GIFs > that we see in Github PRs? >

Re: What tool for GIF webapp previews?

2016-06-15 Thread Federico Valeri
Thank you Corneau. Fortunately it works even under Win7. 2016-06-15 14:48 GMT+02:00 Corneau Damien : > I think most of the Mac OSX users, are using licecap. > (However beware if using multiple monitor, it only record on the main one) > > On Wed, Jun 15, 2016 at 9:47 PM, Federico Valeri > wrote:

[jira] [Created] (ZEPPELIN-1013) Changing Value in Select Form Runs Paragraph Instantly

2016-06-15 Thread Seamus Mc Gonigle (JIRA)
Seamus Mc Gonigle created ZEPPELIN-1013: --- Summary: Changing Value in Select Form Runs Paragraph Instantly Key: ZEPPELIN-1013 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1013 Project: Zepp

[GitHub] zeppelin issue #1016: Bump up version to 0.7.0-SNAPSHOT

2016-06-15 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1016 @minahlee great! Could you also take care of https://github.com/apache/zeppelin/blob/master/docs/_config.yml#L24 https://github.com/apache/zeppelin/blob/master/docs/_config

[GitHub] zeppelin issue #1016: Bump up version to 0.7.0-SNAPSHOT

2016-06-15 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1016 @Leemoonsoo it's already there --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled a

[GitHub] zeppelin issue #1016: Bump up version to 0.7.0-SNAPSHOT

2016-06-15 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1016 Oops i've confused you already took care! thanks! --- 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] zeppelin issue #1016: Bump up version to 0.7.0-SNAPSHOT

2016-06-15 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1016 no problem! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if

[GitHub] zeppelin issue #1016: Bump up version to 0.7.0-SNAPSHOT

2016-06-15 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1016 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] zeppelin issue #1008: [ZEPPELIN-997] Export data to csv

2016-06-15 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1008 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GSoC-2016][Zeppelin Notebooks] Second Notebook Review

2016-06-15 Thread anish singh
Hello, Second notebook is up for review at [0]. Custom visualizations for the notebook were created using html display and Helium Application. I'm Sorry for delay in creating the notebook as filtering and cleaning the datasets for this notebook took a lot of time. Documentation for the notebook i

[GitHub] zeppelin issue #986: [Zeppelin 946] Permissions not honoring group

2016-06-15 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/986 @rja1, Thank you for much detailed analysis. - I have made another commit, this should take care of `makes ldap bindRequest as n...@mydomain.com and fails.` - I cannot do

[jira] [Created] (ZEPPELIN-1014) Blocking the zeppelin-web build error.

2016-06-15 Thread Shim HyungSung (JIRA)
Shim HyungSung created ZEPPELIN-1014: Summary: Blocking the zeppelin-web build error. Key: ZEPPELIN-1014 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1014 Project: Zeppelin Issue T

[GitHub] zeppelin issue #1017: Add force layout visualization in paragraphs for SQL q...

2016-06-15 Thread davidtzoor
Github user davidtzoor commented on the issue: https://github.com/apache/zeppelin/pull/1017 @bzz I have rebased my repo with the latest master. --- 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] zeppelin pull request #1018: Blocking the zeppelin-web build error.

2016-06-15 Thread astroshim
GitHub user astroshim opened a pull request: https://github.com/apache/zeppelin/pull/1018 Blocking the zeppelin-web build error. ### What is this PR for? This PR is for blocking unnecessary zeppelin-web build errors. ``` [ERROR] npm WARN package.json zeppelin-web@0.0.0 No

[GitHub] zeppelin issue #993: [ZEPPELIN-987] Enable user to secure interpreter settin...

2016-06-15 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/993 @echarles Yes. Thanks for your suggestion ! :) --- 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] zeppelin issue #994: [ZEPPELIN-986] Create publish release script

2016-06-15 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/994 @echarles - This script is written exactly because of the reason that @bzz explained above. > calling it multiple times with different arguments, instead of single script was quite e

[GitHub] zeppelin issue #986: [Zeppelin 946] Permissions not honoring group

2016-06-15 Thread rja1
Github user rja1 commented on the issue: https://github.com/apache/zeppelin/pull/986 Nice job @prabhjyotsingh, thanks for your work! I don't see the extra ldap call anymore (step 1). I also don't see the large ldap call any longer (step 4). In addition, all of my roles are mapped

[GitHub] zeppelin issue #994: [ZEPPELIN-986] Create publish release script

2016-06-15 Thread echarles
Github user echarles commented on the issue: https://github.com/apache/zeppelin/pull/994 @minahlee thx for this precision and context. True, I just realized after sending the comment that repo is not part of zeppelin-server. However, I need zeppelin-server in my project (I kin

[GitHub] zeppelin pull request #1010: [ZEPPELIN-998] Extend install.md -> Quick Start

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

[GitHub] zeppelin pull request #1010: [ZEPPELIN-998] Extend install.md -> Quick Start

2016-06-15 Thread AhyoungRyu
GitHub user AhyoungRyu reopened a pull request: https://github.com/apache/zeppelin/pull/1010 [ZEPPELIN-998] Extend install.md -> Quick Start ### What is this PR for? Most of other projects have **Quick Start** or **Getting Started** page for the beginner. Currently, Zeppelin als

[VIRTUAL] Meetup for Data Scientists Tuesday 6/21 - 9AM Pacific : Zeppelin meets MADlib & HAWQ

2016-06-15 Thread Greg Chase
Dear members of the Apache Zeppelin, Apache MADlib, and Apache HAWQ communities, We are hosting a cross-community virtual meeting for data science users this next Tuesday, June 21, 9AM Pacific. No sign up is necessary, just join the event here .

[GitHub] zeppelin issue #994: [ZEPPELIN-986] Create publish release script

2016-06-15 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/994 @echarles If there are such needs, nothing stops us from publishing it. I will make change accordingly. Thank you for sharing your situation. --- If your project is set up for it, you can reply t

[GitHub] zeppelin issue #984: [ZEPPELIN-982] Improve interpreter completion API

2016-06-15 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/984 Thanks @AhyoungRyu for the improvement and taking care of comments. LGTM. Merge it into master and 0.6 branch if there're no more discussions. --- If your project is set up for it, you

[GitHub] zeppelin issue #1013: [ZEPPELIN-1002] Move configuration menus under dropdow...

2016-06-15 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1013 Thanks @minahlee for the update. I think it's less confusing and more organized way to manage menubar. Do you think it'll be merged into branch-0.6, too? On documentation, I think ther

[GitHub] zeppelin issue #836: [WIP] [ZEPPELIN-732] Helium Application

2016-06-15 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/836 It's ready for review. This PR provides experimental Java API to add pluggable components. Which is still i think pretty low level. Adding high level api (e.g. [ZEPPELIN-1003](https

[GitHub] zeppelin issue #1007: Update and refactor NotebookRepo versioning API

2016-06-15 Thread khalidhuseynov
Github user khalidhuseynov commented on the issue: https://github.com/apache/zeppelin/pull/1007 yeah, I agree that having two interfaces is more elegant, but let's see some more detailed downsides. if you see [here](https://github.com/apache/zeppelin/blob/master/zeppelin-zengine/src/m

[GitHub] zeppelin issue #1007: Update and refactor NotebookRepo versioning API

2016-06-15 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1007 Good point, if asked, I would say that having `NotebookRepoSync implements NotebookRepo` is not an elegant design either - NotebookRepoSync is not a NoteboksRepo! :) It's up to you after all,

[GitHub] zeppelin issue #836: [ZEPPELIN-732] Helium Application

2016-06-15 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/836 Great stuff!! It's going to take some time to review. @Leemoonsoo @minahlee Please help me understand, do you think this should be included in 0.6.0 release? As I do not see [ZEPPELIN-732](ht

[GitHub] zeppelin issue #1013: [ZEPPELIN-1002] Move configuration menus under dropdow...

2016-06-15 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1013 I think it is common practice to have settings under profile dropdown menu. ### Examples **Github** https://cloud.githubusercontent.com/assets/8503346/16098300/2999a8e0-3307-11e6-8

[GitHub] zeppelin issue #1013: [ZEPPELIN-1002] Move configuration menus under dropdow...

2016-06-15 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1013 Just FYI - please mind that second menu should play nice with mobile layout. As first one is collapsed on mobile, even right now with just a search search - it sometimes is not enough space

[GitHub] zeppelin issue #836: [ZEPPELIN-732] Helium Application

2016-06-15 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/836 @khalidhuseynov Just FYI, this PR adds 4 new messages websocket protocol: ``` APP_APPEND_OUTPUT, // [s-c] append output APP_UPDATE_OUTPUT, // [s-c] update (replace) output

[GitHub] zeppelin issue #836: [ZEPPELIN-732] Helium Application

2016-06-15 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/836 @bzz Pluggable visualization is one of [roadmap of 0.6 release](https://cwiki.apache.org/confluence/display/ZEPPELIN/Zeppelin+Roadmap), and i think this PR provides a way to enable it.

[jira] [Created] (ZEPPELIN-1015) Cron job fails to run a paragraph when multiple type of interpreter is being used

2016-06-15 Thread Lee moon soo (JIRA)
Lee moon soo created ZEPPELIN-1015: -- Summary: Cron job fails to run a paragraph when multiple type of interpreter is being used Key: ZEPPELIN-1015 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1015

[GitHub] zeppelin pull request #1019: [ZEPPELIN-1015] Cron job fails to run a paragra...

2016-06-15 Thread Leemoonsoo
GitHub user Leemoonsoo opened a pull request: https://github.com/apache/zeppelin/pull/1019 [ZEPPELIN-1015] Cron job fails to run a paragraph when multiple type of interpreter is being used ### What is this PR for? Cron job can fail when notebook uses multiple types of paragraphs

[GitHub] zeppelin pull request #1020: [HOTFIX][DOC] Fix image path

2016-06-15 Thread minahlee
GitHub user minahlee opened a pull request: https://github.com/apache/zeppelin/pull/1020 [HOTFIX][DOC] Fix image path ### What is this PR for? Use relative path instead of absolute path so doc can find image under proper version folder ### What type of PR is it? Doc

[GitHub] zeppelin issue #1016: Bump up version to 0.7.0-SNAPSHOT

2016-06-15 Thread darionyaphet
Github user darionyaphet commented on the issue: https://github.com/apache/zeppelin/pull/1016 Have any plan to release 0.6 ? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabl

[GitHub] zeppelin issue #1017: Add force layout visualization in paragraphs for SQL q...

2016-06-15 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1017 @davidtzoor didn't go well. Could you try to have your changes in a different branch and re-create a PR? --- If your project is set up for it, you can reply to this email and have your repl

[GitHub] zeppelin issue #1013: [ZEPPELIN-1002] Move configuration menus under dropdow...

2016-06-15 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1013 @bzz actually, last time I tested, the about dropdown was doing great on mobile @minahlee I still prefer separated one, mainly because you won't always have a user, and having a dr

[GitHub] zeppelin issue #1020: [HOTFIX][DOC] Fix image path

2016-06-15 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1020 👍 looks great, let's 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 have this feature enabled and w

[GitHub] zeppelin pull request #1021: Python interpreter and doc cleanup

2016-06-15 Thread bzz
GitHub user bzz opened a pull request: https://github.com/apache/zeppelin/pull/1021 Python interpreter and doc cleanup ### What is this PR for? This is first step improving current Python interpreter implementation. It has just a cleanup, style and docs improvements.

[GitHub] zeppelin issue #1018: Blocking the zeppelin-web build error.

2016-06-15 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1018 Is it a build WARNING or an actual error? Asking as have not seen CI build failure due to this error before --- If your project is set up for it, you can reply to this email and have your re

[GitHub] zeppelin issue #1016: Bump up version to 0.7.0-SNAPSHOT

2016-06-15 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1016 @minahlee looks great, let's merge it @darionyaphet all release-related question are discussed on dev@ mailing list and tracked under [ZEPPELIN-889](https://issues.apache.org/jira/browse/ZEPP

[GitHub] zeppelin issue #1014: Remove Incubator and fix typos in README

2016-06-15 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1014 Let's merge the guy if there is no further discussion --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] zeppelin issue #1018: Blocking the zeppelin-web build error.

2016-06-15 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1018 It's not actual error just error messsage. --- 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] zeppelin issue #1013: [ZEPPELIN-1002] Move configuration menus under dropdow...

2016-06-15 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1013 > @bzz actually, last time I tested, the about dropdown was doing great on mobile Have you already tested 2 proposed dropdowns on mobile? If possible, plz share the screenshots --- If your

[GitHub] zeppelin issue #1021: Python interpreter and doc cleanup

2016-06-15 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1021 CI failure is not related and is taken care of under #1015 ``` Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 34.3 sec - in org.apache.zeppelin.integration.SparkParagraph

[GitHub] zeppelin pull request #1020: [HOTFIX][DOC] Fix image path

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

[GitHub] zeppelin issue #1020: [HOTFIX][DOC] Fix image path

2016-06-15 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1020 I feel like we battled against the opposite case before: https://github.com/apache/zeppelin/pull/899 --- If your project is set up for it, you can reply to this email and have your reply ap

[GitHub] zeppelin pull request #1022: [HOTFIX][DOC] Fix image path again

2016-06-15 Thread minahlee
GitHub user minahlee opened a pull request: https://github.com/apache/zeppelin/pull/1022 [HOTFIX][DOC] Fix image path again ### What is this PR for? Realized that #1020 doesn't do the job. This PR does fix the issue of broken image on website and already documented on the websit

[GitHub] zeppelin issue #1022: [HOTFIX][DOC] Fix image path again

2016-06-15 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1022 Yes, we fixed the `/assets` type of links a few times before. The best way to check the image link it to use the 'sources' tab in the developer tools to see the file tree. https://c

[GitHub] zeppelin issue #1022: [HOTFIX][DOC] Fix image path again

2016-06-15 Thread minahlee
Github user minahlee commented on the issue: https://github.com/apache/zeppelin/pull/1022 Thank you for the tip! wont make same mistake 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 not hav

[GitHub] zeppelin issue #1018: Blocking the zeppelin-web build error.

2016-06-15 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1018 It's not an ERROR and doesn't break the build. It's more of a warning/info but when running it is shown with [ERROR] which can be misleading when looking for build errors. I actuall

[GitHub] zeppelin issue #1018: Blocking the zeppelin-web build error.

2016-06-15 Thread corneadoug
Github user corneadoug commented on the issue: https://github.com/apache/zeppelin/pull/1018 However some of those errors are more related to deprecated versions and might need more tests if we update the dependencies. Maybe it should be done in a different PR. --- If your project

[GitHub] zeppelin issue #1012: ZEPPELIN-999 Support alias for JDBC properties

2016-06-15 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1012 I also made a PR as a generalized version. https://issues.apache.org/jira/browse/ZEPPELIN-1012 --- If your project is set up for it, you can reply to this email and have your reply appear on Git

[GitHub] zeppelin issue #986: [Zeppelin 946] Permissions not honoring group

2016-06-15 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue: https://github.com/apache/zeppelin/pull/986 @rja1 Good to know, I'm able to resolve what you asked for. --- 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 do

[GitHub] zeppelin pull request #1023: [gh-pages][ZEPPELIN-996] Update first page of Z...

2016-06-15 Thread AhyoungRyu
GitHub user AhyoungRyu opened a pull request: https://github.com/apache/zeppelin/pull/1023 [gh-pages][ZEPPELIN-996] Update first page of Zeppelin website ### What is this PR for? Apply same first page(`index.html`) to `gh-pages`. ### What type of PR is it? Documentat

[GitHub] zeppelin pull request #1012: ZEPPELIN-999 Support alias for JDBC properties

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

  1   2   >