Skip to content

Conversation

@tedyu
Copy link
Contributor

@tedyu tedyu commented Oct 26, 2015

No description provided.

@tedyu tedyu changed the title [DOC] Include hive profile in make-distribution.sh command SPARK-11318 [DOC] Include hive profile in make-distribution.sh command Oct 26, 2015
@SparkQA
Copy link

SparkQA commented Oct 26, 2015

Test build #44361 has finished for PR 9281 at commit 2ad7393.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@srowen
Copy link
Member

srowen commented Oct 26, 2015

I see the point, but, this is also not how the release tarballs are build. Look at release-build.sh. I wouldn't mind adding more of the flags you see in the releases, but this is also not advertised as something that creates exactly what's in a release tarball. There is no one such release, and if there is a "main" one, it's hadoop-provided at this point.

@tedyu tedyu changed the title SPARK-11318 [DOC] Include hive profile in make-distribution.sh command SPARK-11318 Include hive profile in make-distribution.sh command Oct 26, 2015
@SparkQA
Copy link

SparkQA commented Oct 26, 2015

Test build #44372 has finished for PR 9281 at commit a12997b.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@srowen
Copy link
Member

srowen commented Oct 26, 2015

LGTM

@tedyu
Copy link
Contributor Author

tedyu commented Oct 27, 2015

Can this be merged ?

Thanks

@asfgit asfgit closed this in f304f9c Oct 29, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants