[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

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

2015-11-05 Thread pfcurtis
Github user pfcurtis commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/391#issuecomment-154067574 Added MapR specific Maven command lines to the README.md --- 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: Corrected and updated the MapR Di...

2015-11-04 Thread pfcurtis
GitHub user pfcurtis opened a pull request: https://github.com/apache/incubator-zeppelin/pull/391 Corrected and updated the MapR Distribution profiles Completed the necessary Spark dependency profiles for the MapR distribution. Also added commented out MapR artifact IDs for the

[GitHub] incubator-zeppelin pull request: MapR 4 Profile corrected, Added A...

2015-10-30 Thread pfcurtis
Github user pfcurtis commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/110#issuecomment-152637145 I'm closing this pull request, and re-opening a new one with only the MapR profile updates. --- If your project is set up for it, you can reply to

[GitHub] incubator-zeppelin pull request: MapR 4 Profile corrected, Added A...

2015-10-30 Thread pfcurtis
Github user pfcurtis closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/110 --- 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: MapR 4 Profile corrected, Added A...

2015-10-25 Thread pfcurtis
Github user pfcurtis commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/110#issuecomment-150920846 I will rework the pull request for the MapR profiles, and drop the code for the Drill interpreter. paul On 10/25/15 8:57 AM, Jongyoul

[GitHub] incubator-zeppelin pull request: MapR 4 Profile corrected, Added A...

2015-10-15 Thread pfcurtis
Github user pfcurtis commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/110#issuecomment-148391010 I think it is a good idea to split the profiles from the Drill interpreter. As the Drill interpreter is simply a JDBC connection, that should be almost no

[GitHub] incubator-zeppelin pull request: MapR 4 Profile corrected, Added A...

2015-09-23 Thread pfcurtis
Github user pfcurtis commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/110#issuecomment-142544686 @WalterDalton: I agree. Let me split this into two pull requests. The reason the Apache Drill PR hasn't been merged is I haven't had the time to

[GitHub] incubator-zeppelin pull request: MapR 4 Profile corrected, Added A...

2015-07-08 Thread pfcurtis
Github user pfcurtis commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/110#issuecomment-119775837 Fetched the upstream master and checked that everything still compiles and executes properly. I think this (and the Drill interpreter) are ready to merge

[GitHub] incubator-zeppelin pull request: MapR 4 Profile corrected, Added A...

2015-07-03 Thread pfcurtis
Github user pfcurtis commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/110#issuecomment-118433095 https://issues.apache.org/jira/browse/DRILL-3329 --- 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: MapR 4 Profile corrected, Added A...

2015-07-02 Thread pfcurtis
Github user pfcurtis commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/110#issuecomment-118049174 I've removed the incorrect POM file that was created during packaging. --- If your project is set up for it, you can reply to this email and have

[GitHub] incubator-zeppelin pull request: MapR 4 Profile corrected, Added A...

2015-06-20 Thread pfcurtis
Github user pfcurtis commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/110#issuecomment-113835409 On 6/20/15 6:44 PM, Lee moon soo wrote: > > Thanks for making pullrequest! > > It looks like org.apache.drill:drill-jdbc-all:ja

[GitHub] incubator-zeppelin pull request: MapR 4 Profile corrected, Added A...

2015-06-19 Thread pfcurtis
GitHub user pfcurtis opened a pull request: https://github.com/apache/incubator-zeppelin/pull/110 MapR 4 Profile corrected, Added Apache Drill Interpreter * Corrected the "mapr4" profile to correctly pull the Maven artifacts from the MapR Maven repository. Added the MapR