You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zeppelin.apache.org by pfcurtis <gi...@git.apache.org> on 2015/11/04 21:04:04 UTC
[GitHub] incubator-zeppelin pull request: Corrected and updated the MapR Di...
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 Hive interpreter.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/pfcurtis/incubator-zeppelin master
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-zeppelin/pull/391.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #391
----
commit 7ed07a0ba82fa79911f5d0e328fb5c9930b03f14
Author: Paul Curtis <pc...@mapr.com>
Date: 2015-11-04T19:59:47Z
Corrected and updated the MapR Distribution profiles
----
---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: Corrected and updated the MapR Di...
Posted by Leemoonsoo <gi...@git.apache.org>.
Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/391#issuecomment-154085908
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 feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
Re: [GitHub] incubator-zeppelin pull request: Corrected and updated
the MapR Di...
Posted by Paul Curtis <pc...@maprtech.com>.
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
using Zeppelin.
Regards.
On Tue, Nov 10, 2015 at 6:15 PM, Leemoonsoo <gi...@git.apache.org> 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?
>
>
> ---
> 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 enabled but not working, please
> contact infrastructure at infrastructure@apache.org or file a JIRA ticket
> with INFRA.
> ---
>
--
*Paul Curtis *- Field Enablement Team
*O: *+1 203-660-0015 - *M:* +1 203-539-9705
Now Available - Free Hadoop On-Demand Training
<http://www.mapr.com/training?utm_source=Email&utm_medium=Signature&utm_campaign=Free%20available>
Re: [GitHub] incubator-zeppelin pull request: Corrected and updated
the MapR Di...
Posted by Paul Curtis <pc...@maprtech.com>.
Done and pushed.
Regards.
On Tue, Nov 10, 2015 at 6:15 PM, Leemoonsoo <gi...@git.apache.org> 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?
>
>
> ---
> 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 enabled but not working, please
> contact infrastructure at infrastructure@apache.org or file a JIRA ticket
> with INFRA.
> ---
>
--
*Paul Curtis *- Field Enablement Team
*O: *+1 203-660-0015 - *M:* +1 203-539-9705
Now Available - Free Hadoop On-Demand Training
<http://www.mapr.com/training?utm_source=Email&utm_medium=Signature&utm_campaign=Free%20available>
[GitHub] incubator-zeppelin pull request: Corrected and updated the MapR Di...
Posted by Leemoonsoo <gi...@git.apache.org>.
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 appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: Corrected and updated the MapR Di...
Posted by pfcurtis <gi...@git.apache.org>.
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 as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: Corrected and updated the MapR Di...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/391
---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: Corrected and updated the MapR Di...
Posted by Leemoonsoo <gi...@git.apache.org>.
Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/391#issuecomment-153956322
Looks good. Do you mind add simple description for the profiles (mapr40, mapr50) for users? README.md could be one place.
---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: Corrected and updated the MapR Di...
Posted by pfcurtis <gi...@git.apache.org>.
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.
Regards.
On 11/10/2015 07:41 AM, Lee moon soo wrote:
>
> According to https://issues.apache.org/jira/browse/ZEPPELIN-404, MapR maven repository shouldn't be activated by default. Could you move MapR repository into profile, like #404 <https://github.com/apache/incubator-zeppelin/pull/404> did for cloudera repo?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/apache/incubator-zeppelin/pull/391#issuecomment-155410179>.
>
---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: Corrected and updated the MapR Di...
Posted by Leemoonsoo <gi...@git.apache.org>.
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 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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: Corrected and updated the MapR Di...
Posted by Leemoonsoo <gi...@git.apache.org>.
Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/391#issuecomment-155052776
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 your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-zeppelin pull request: Corrected and updated the MapR Di...
Posted by Leemoonsoo <gi...@git.apache.org>.
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 into profile, like https://github.com/apache/incubator-zeppelin/pull/404 did for cloudera repo?
---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---