Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SPARK-6196] [BUILD] Remove MapR profiles in favor of hadoop-provided #8338

Closed
wants to merge 1 commit into from

Conversation

srowen
Copy link
Member

@srowen srowen commented Aug 20, 2015

Follow up to #7047

@pwendell mentioned that MapR should use hadoop-provided now, and indeed the new build script does not produce mapr3/mapr4 artifacts anymore. Hence the action seems to be to remove the profiles, which are now not used.

CC @trystanleftwich

@SparkQA
Copy link

SparkQA commented Aug 20, 2015

Test build #41319 timed out for PR 8338 at commit 14bcab4 after a configured wait of 175m.

@SparkQA
Copy link

SparkQA commented Aug 20, 2015

Test build #1675 timed out for PR 8338 at commit 14bcab4 after a configured wait of 175m.

@rxin
Copy link
Contributor

rxin commented Aug 21, 2015

I will let @pwendell respond to this when he comes back from vacation.

@srowen
Copy link
Member Author

srowen commented Aug 21, 2015

@rxin yeah that's fine, there's no emergency, although these profiles are already unused; the release script for 1.5 will not produce artifacts that use these. On the other PR Patrick indicated, I think, that he intends these to be removed in favor of hadoop-provided

@pwendell
Copy link
Contributor

Yes this LGTM - these are outdated and I don't even think MapR is advising their customers to use these. They are asking people to use hadoop-provided, which was created to simplify using Spark with different MapR versions.

asfgit pushed a commit that referenced this pull request Aug 25, 2015
Follow up to #7047

pwendell mentioned that MapR should use `hadoop-provided` now, and indeed the new build script does not produce `mapr3`/`mapr4` artifacts anymore. Hence the action seems to be to remove the profiles, which are now not used.

CC trystanleftwich

Author: Sean Owen <sowen@cloudera.com>

Closes #8338 from srowen/SPARK-6196.

(cherry picked from commit 57b960b)
Signed-off-by: Sean Owen <sowen@cloudera.com>
@asfgit asfgit closed this in 57b960b Aug 25, 2015
@srowen
Copy link
Member Author

srowen commented Aug 25, 2015

Nice, I do agree with that, and think it will be a better experience for all. The vendor <repository> declarations can stay as a convenience. I'd be inclined to do something similar for the cdh4 profile. I can see that it functions as a sort of generic hadoop-2.0.0 build but would not mind zapping it.

@srowen srowen deleted the SPARK-6196 branch August 25, 2015 07:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants