Skip to content

[BUILD] Remove dependency reduced POM hack #7919

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.

Sign up for GitHub

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

Jump to bottom
Closed
tedyu wants to merge 1 commit into from
Closed

[BUILD] Remove dependency reduced POM hack #7919

tedyu wants to merge 1 commit into from

Conversation

@tedyu
Copy link
Contributor

@tedyu tedyu commented Aug 4, 2015

This PR removes the dependency reduced POM hack brought back by #7191

@SparkQA
Copy link

SparkQA commented Aug 4, 2015

Test build #39633 has finished for PR 7919 at commit 1bfbd7b.

  • This patch fails PySpark unit tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • public static final class SortedIterator extends UnsafeSorterIterator
    • public class KVSorterIterator extends KVIterator<UnsafeRow, UnsafeRow>

@tedyu
Copy link
Contributor Author

tedyu commented Aug 4, 2015

The PySpark test failure should not be related to the change.

@srowen
Copy link
Member

srowen commented Aug 4, 2015

This is also related to #7826
Yes, I agree this can't be related. The 'test' of this line being removed has already passed in this PR.
LGTM, will merge soon.

asfgit pushed a commit that referenced this pull request Aug 4, 2015
@tedyu @srowen
[SPARK-8064] [BUILD] Follow-up. Undo change from SPARK-9507 that was …
bd9b752 
…accidentally reverted

This PR removes the dependency reduced POM hack brought back by #7191

Author: tedyu <yuzhihong@gmail.com>

Closes #7919 from tedyu/master and squashes the following commits:

1bfbd7b [tedyu] [BUILD] Remove dependency reduced POM hack

(cherry picked from commit b211cbc)
Signed-off-by: Sean Owen <sowen@cloudera.com>
@asfgit asfgit closed this in b211cbc Aug 4, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@tedyu @SparkQA @srowen