Skip to content

Conversation

@sameeragarwal
Copy link
Member

What changes were proposed in this pull request?

Makes UnsafeSortDataFormat and RecordPointerAndKeyPrefix public. These are already public in 2.0 and are used in an ExternalSorterSuite test (see 0b8bdf7)

How was this patch tested?

Successfully builds locally

@sameeragarwal
Copy link
Member Author

cc @rxin

@rxin
Copy link
Contributor

rxin commented May 26, 2016

LGTM pending tests.

@SparkQA
Copy link

SparkQA commented May 27, 2016

Test build #59427 has finished for PR 13339 at commit dbf752b.

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

@rxin
Copy link
Contributor

rxin commented May 27, 2016

Merging in 1.6.

asfgit pushed a commit that referenced this pull request May 27, 2016
## What changes were proposed in this pull request?

Makes `UnsafeSortDataFormat`  and `RecordPointerAndKeyPrefix` public. These are already public in 2.0 and are used in an `ExternalSorterSuite` test (see 0b8bdf7)

## How was this patch tested?

Successfully builds locally

Author: Sameer Agarwal <[email protected]>

Closes #13339 from sameeragarwal/fix-compile.
zzcclp pushed a commit to zzcclp/spark that referenced this pull request May 27, 2016
## What changes were proposed in this pull request?

Makes `UnsafeSortDataFormat`  and `RecordPointerAndKeyPrefix` public. These are already public in 2.0 and are used in an `ExternalSorterSuite` test (see apache@0b8bdf7)

## How was this patch tested?

Successfully builds locally

Author: Sameer Agarwal <[email protected]>

Closes apache#13339 from sameeragarwal/fix-compile.

(cherry picked from commit c53c83c)
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