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

2016-04-27 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/858#issuecomment-215308849 Right, it is supported only in pro ver. Handsontable's Export feature is in front-end. IMHO export should be a backend feature and will be a good addition

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-209] Folder support for...

2016-04-27 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/796#issuecomment-215308148 @johnnyws Tested and working 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.

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-798] Migrate to Jetty v...

2016-04-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/831 --- 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: ZEPPELIN-824 - Make CI Crash when...

2016-04-27 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/854 --- 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: [ZEPPELIN-672] Add feature to exp...

2016-04-27 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/761#issuecomment-215292663 @felixcheung already saw it and I will check it. However, while we test + discuss the other PR, there is no reason not to merge this one. If the

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-209] Folder support for...

2016-04-27 Thread johnnyws
Github user johnnyws commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/796#issuecomment-215241100 @Leemoonsoo I've fixed the issue and rebased with master --- If your project is set up for it, you can reply to this email and have your reply appear on

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

2016-04-27 Thread prasadwagle
GitHub user prasadwagle opened a pull request: https://github.com/apache/incubator-zeppelin/pull/860 [ZEPPELIN-828] Use individual user credentials for data source authentication ### What is this PR for? The purpose of this pull request is to allow users to use their own

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-183] Interpreter proces...

2016-04-27 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/795#issuecomment-215226858 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

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

2016-04-27 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/858#issuecomment-215225519 We would need export to file feature which seems like is not supported in the free Handsontable version? --- If your project is set up for it, you can

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-157] Adding Map Visuali...

2016-04-27 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/765#issuecomment-215224988 @Madhuka let us know if we could help! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-599]notebook search sho...

2016-04-27 Thread ravicodder
Github user ravicodder commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/859#issuecomment-215164810 Ready for review. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-599]notebook search sho...

2016-04-27 Thread ravicodder
GitHub user ravicodder opened a pull request: https://github.com/apache/incubator-zeppelin/pull/859 [ZEPPELIN-599]notebook search should search paragraph title ### What is this PR for? Allow notebook search to search paragraph title too. ### What type of PR is it?

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-183] Interpreter proces...

2016-04-27 Thread vijikarthi
Github user vijikarthi commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/795#issuecomment-215142318 +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

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-183] Interpreter proces...

2016-04-27 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/795#issuecomment-215135830 I'm 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 as

[GitHub] incubator-zeppelin pull request: Improve README.md

2016-04-27 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/839#issuecomment-215135400 I think it would be better if both -Psparkr and -Pr are included with some description for them --- If your project is set up for it, you can reply to

[GitHub] incubator-zeppelin pull request: add /r/lib/ in .gitignore

2016-04-27 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/857#issuecomment-215134558 +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

[GitHub] incubator-zeppelin pull request: add /r/lib/ in .gitignore

2016-04-27 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/857#issuecomment-215134208 Merging if no more discussion. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-672] Add feature to exp...

2016-04-27 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/761#issuecomment-215126421 Could you check out #858? Seems to overlap with this PR https://github.com/apache/incubator-zeppelin/pull/858 --- If

[GitHub] incubator-zeppelin pull request: ZEPPELIN-157: Adding Map Visualiz...

2016-04-27 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/152#issuecomment-215126055 Great - will you have time to get that ready then? _ --- If your project is set up

Re: Zeppelin Helium Build failure

2016-04-27 Thread moon soo Lee
Hi Anish, Thanks for the interest. Could you try build https://github.com/apache/incubator-zeppelin/pull/836 instead of https://github.com/Leemoonsoo/incubator-zeppelin/tree/helium? #836 is still work in progress but implemented most of proposed features of ProofOfConcept implementation.

Zeppelin Helium Build failure

2016-04-27 Thread anish singh
Hello, I tried to build the Zeppelin project from the Helium branch as specified at [0]. The command specified was(inside the Zeppelin directory): mvn clean package -DskipTests The build failed. The log for the failure is below, please provide help in how to build the project error log:-

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

2016-04-27 Thread r-kamath
Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/858#issuecomment-215040456 Won't affect #714 and #725 #6 and #761 are using jQuery Datatables This PR is a proposal to use Handsontable instead of datatables.

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-672] Add feature to exp...

2016-04-27 Thread corneadoug
Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/761#issuecomment-215016412 Merging this --- 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-157: Adding Map Visualiz...

2016-04-27 Thread Madhuka
Github user Madhuka commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/152#issuecomment-214997716 Referee #765 --- 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-157: Adding Map Visualiz...

2016-04-27 Thread felixcheung
Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/152#issuecomment-214995913 Based on the earlier comment, are we moving to #728 then? --- If your project is set up for it, you can reply to

[GitHub] incubator-zeppelin pull request: ZEPPELIN-804 Refactoring registra...

2016-04-27 Thread jongyoul
Github user jongyoul closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/835 --- 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: ZEPPELIN-804 Refactoring registra...

2016-04-27 Thread jongyoul
GitHub user jongyoul reopened a pull request: https://github.com/apache/incubator-zeppelin/pull/835 ZEPPELIN-804 Refactoring registration mechanism on Interpreters ### What is this PR for? This PR enable Zeppelin server register Interpreters without any dependencies of their

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-183] Interpreter proces...

2016-04-27 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/795#issuecomment-214979591 @vijikarthi Thanks for testing. 'SPARK-10872' is expected while default configuration of sparkcontext creates 'metastore_db' in the same location. If you