[GitHub] incubator-zeppelin pull request: [ZEPPELIN-554] Streaming interpre...

2016-01-08 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/611#issuecomment-170203990 @Leemoonsoo this is so awesome and useful feature. Thanks a lot of this. --- If your project is set up for it, you can reply to this email and have yo

[GitHub] incubator-zeppelin pull request: fix + paragraph appearing in link

2016-01-08 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/613#issuecomment-170195136 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: Shell interpreter improvements

2016-01-08 Thread Karuppayya
Github user Karuppayya closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/615 --- 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: Rest Apis to export/import

2016-01-08 Thread swakrish
Github user swakrish commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/614#issuecomment-170189394 @bzz thanks for your comments. I did look at the UI functions, it doesnt have an import I guess and export is more or less similar. Let me know your thought

[GitHub] incubator-zeppelin pull request: Shell interpreter improvements

2016-01-08 Thread Karuppayya
GitHub user Karuppayya opened a pull request: https://github.com/apache/incubator-zeppelin/pull/615 Shell interpreter improvements ### What is this PR for? *Provide ability to to run shell commands in parallel *Provide ability to cancel shell command *Propagate the error

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

2016-01-08 Thread bzz
Github user bzz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/614#issuecomment-170185380 Thank you for improvement! Reconciling the exisiting GUI import/export features to use this API may be a good idea too, otherwise we'll have different co

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-572] pyspark interprete...

2016-01-08 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/605#issuecomment-170175820 Sure. --- 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 fea

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

2016-01-08 Thread swakrish
Github user swakrish commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/614#issuecomment-170173625 I added them --- 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: [ZEPPELIN-572] pyspark interprete...

2016-01-08 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/605#issuecomment-170172256 If there're no more discussions, i'm merging it into 'branch-0.5.6' and 'master' --- If your project is set up for it, you can reply to this email and ha

[GitHub] incubator-zeppelin pull request: Hotfix: Update maven-gpg-plugin a...

2016-01-08 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/612 --- 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 + paragraph appearing in link

2016-01-08 Thread AhyoungRyu
Github user AhyoungRyu commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/613#issuecomment-170171394 Thank you for the quick fix !! 😀 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If you

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

2016-01-08 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/614#issuecomment-170170069 Thanks @swakrish for the contribution! Do you mind update `docs/rest-api/rest-notebook.md`, too? Changing this file updates zeppelin website. --- I

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

2016-01-08 Thread swakrish
Github user swakrish commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/614#issuecomment-170169529 https://cloud.githubusercontent.com/assets/7673174/12212982/d4d6c7a6-b625-11e5-9307-3d49603a8b8e.png";> attaching screenshots for import Junit --- If yo

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

2016-01-08 Thread swakrish
Github user swakrish commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/614#issuecomment-170169471 https://cloud.githubusercontent.com/assets/7673174/12212976/bc53191e-b625-11e5-8753-13ff3615b9a5.png";> attaching screenshot for export JUnits --- If yo

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

2016-01-08 Thread swakrish
GitHub user swakrish opened a pull request: https://github.com/apache/incubator-zeppelin/pull/614 Rest Apis to export/import 1) Rest apis to export notebook as JSON 2) Rest api to import notebook from JSON https://issues.apache.org/jira/browse/ZEPPELIN-590 You can merge t

[GitHub] incubator-zeppelin pull request: fix + paragraph appearing in link

2016-01-08 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/613#issuecomment-170167859 Thanks @MikeTYChen for really quick fix. Tested and Looks good to me. If there're no more discussions, I'm merging it into both 'master' and 'branch-0.

[GitHub] incubator-zeppelin pull request: fix + paragraph appearing in link

2016-01-08 Thread MikeTYChen
GitHub user MikeTYChen opened a pull request: https://github.com/apache/incubator-zeppelin/pull/613 fix + paragraph appearing in link ### What is this PR for? Fix a bug where the + paragraph appears in link paragraph ### What type of PR is it? Bug Fix ### To

[GitHub] incubator-zeppelin pull request: JDBC generic interpreter

2016-01-08 Thread vgmartinez
Github user vgmartinez commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/361#issuecomment-170147338 I removed the dependencies and add the postgres dependency to have it by default without having to add the jar to the classpath. What do you think? ---

[jira] [Created] (ZEPPELIN-591) '+' sign is shown up together, when click "Link this paragraph"

2016-01-08 Thread Ahyoung (JIRA)
Ahyoung created ZEPPELIN-591: Summary: '+' sign is shown up together, when click "Link this paragraph" Key: ZEPPELIN-591 URL: https://issues.apache.org/jira/browse/ZEPPELIN-591 Project: Zeppelin

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-554] Streaming interpre...

2016-01-08 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/611#issuecomment-170125566 @prabhjyotsingh @corneadoug Addressed comment ![stream_output](https://cloud.githubusercontent.com/assets/1540981/12209696/b45f1634-b609-11e5-9a53-7f98

[jira] [Created] (ZEPPELIN-590) Zeppelin REST api to import notebooks

2016-01-08 Thread Ramaswamy Devarajan (JIRA)
Ramaswamy Devarajan created ZEPPELIN-590: Summary: Zeppelin REST api to import notebooks Key: ZEPPELIN-590 URL: https://issues.apache.org/jira/browse/ZEPPELIN-590 Project: Zeppelin I

[GitHub] incubator-zeppelin pull request: JDBC generic interpreter

2016-01-08 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/361#issuecomment-170056833 I think postgresql is good candidate to be default, too. Leaving it undecided also good to me. --- If your project is set up for it, you can reply to thi

[GitHub] incubator-zeppelin pull request: Hotfix: Update maven-gpg-plugin a...

2016-01-08 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/612#issuecomment-170055316 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-572] pyspark interprete...

2016-01-08 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/605#issuecomment-170045920 Although release branch 'branch-0.5.6' is created by https://issues.apache.org/jira/browse/ZEPPELIN-567, I think this change worth to apply 0.5.6. Shell w

[GitHub] incubator-zeppelin pull request: Hotfix: Update maven-gpg-plugin a...

2016-01-08 Thread bzz
GitHub user bzz opened a pull request: https://github.com/apache/incubator-zeppelin/pull/612 Hotfix: Update maven-gpg-plugin and remove things failing deploy This PR updates plugin GPG version as well as fix `mvn deploy` to play nice again with the release plugin. This will

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-572] pyspark interprete...

2016-01-08 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/605#issuecomment-170033402 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: JDBC generic interpreter

2016-01-08 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/361#issuecomment-170033129 @vgmartinez Thanks for making it clear. You look like that you want to includes some jdbc connectors but it's not appropriate because we don't decide which

[GitHub] incubator-zeppelin pull request: ZEPPELIN-346 Fix NotebookTest not...

2016-01-08 Thread RPCMoritz
Github user RPCMoritz commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/347#issuecomment-170016876 testSelectingReplImplementation(org.apache.zeppelin.notebook.NotebookTest) Time elapsed: 0.032 sec <<< ERROR! org.apache.zeppelin.interpreter.Interpre

[jira] [Created] (ZEPPELIN-589) Test search functionality

2016-01-08 Thread Prabhjyot Singh (JIRA)
Prabhjyot Singh created ZEPPELIN-589: Summary: Test search functionality Key: ZEPPELIN-589 URL: https://issues.apache.org/jira/browse/ZEPPELIN-589 Project: Zeppelin Issue Type: Sub-task

[GitHub] incubator-zeppelin pull request: Shiro security v2

2016-01-08 Thread hayssams
Github user hayssams commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/586#issuecomment-169953798 @Leemoonsoo Great news. I had great pleasure working on it and take into account all your remarks. I'll now start to write down a proposal on how Z

[jira] [Created] (ZEPPELIN-588) Test functionality of Markdown

2016-01-08 Thread Prabhjyot Singh (JIRA)
Prabhjyot Singh created ZEPPELIN-588: Summary: Test functionality of Markdown Key: ZEPPELIN-588 URL: https://issues.apache.org/jira/browse/ZEPPELIN-588 Project: Zeppelin Issue Type: Sub-t

[jira] [Created] (ZEPPELIN-587) Test functionality of spark, pyspark, sparksql

2016-01-08 Thread Prabhjyot Singh (JIRA)
Prabhjyot Singh created ZEPPELIN-587: Summary: Test functionality of spark, pyspark, sparksql Key: ZEPPELIN-587 URL: https://issues.apache.org/jira/browse/ZEPPELIN-587 Project: Zeppelin I

[jira] [Created] (ZEPPELIN-586) Increase test coverage for selenium

2016-01-08 Thread Prabhjyot Singh (JIRA)
Prabhjyot Singh created ZEPPELIN-586: Summary: Increase test coverage for selenium Key: ZEPPELIN-586 URL: https://issues.apache.org/jira/browse/ZEPPELIN-586 Project: Zeppelin Issue Type:

[GitHub] incubator-zeppelin pull request: Shiro security v2

2016-01-08 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/586#issuecomment-169941061 @hayssams Thanks for explain about auth mode. Really appreciate for this great contribution and such a long patience. I think this is really good first

[GitHub] incubator-zeppelin pull request: JDBC generic interpreter

2016-01-08 Thread vgmartinez
Github user vgmartinez commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/361#issuecomment-169939310 Hi @jongyoul, Is rebase now...let me know if you need help with testing or anything else. --- If your project is set up for it, you can reply to this

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-554] Streaming interpre...

2016-01-08 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/611#issuecomment-169927695 Made a PR to refactor the JS code :) --- 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