Skip to content

Conversation

@loneknightpy
Copy link
Contributor

@loneknightpy loneknightpy commented Oct 18, 2016

What changes were proposed in this pull request?

Fix hadoop2.2 compilation error.

How was this patch tested?

Existing tests.

cc @tdas @zsxwing

@zsxwing
Copy link
Member

zsxwing commented Oct 18, 2016

LGTM pending tests

@SparkQA
Copy link

SparkQA commented Oct 19, 2016

Test build #67146 has finished for PR 15537 at commit fd0b076.

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

@zsxwing
Copy link
Member

zsxwing commented Oct 19, 2016

Thanks! Merging to master and 2.0.

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

Fix hadoop2.2 compilation error.

## How was this patch tested?

Existing tests.

cc tdas zsxwing

Author: Yu Peng <[email protected]>

Closes #15537 from loneknightpy/fix-17711.

(cherry picked from commit 2629cd7)
Signed-off-by: Shixiong Zhu <[email protected]>
@asfgit asfgit closed this in 2629cd7 Oct 19, 2016
robert3005 pushed a commit to palantir/spark that referenced this pull request Nov 1, 2016
## What changes were proposed in this pull request?

Fix hadoop2.2 compilation error.

## How was this patch tested?

Existing tests.

cc tdas zsxwing

Author: Yu Peng <[email protected]>

Closes apache#15537 from loneknightpy/fix-17711.
uzadude pushed a commit to uzadude/spark that referenced this pull request Jan 27, 2017
## What changes were proposed in this pull request?

Fix hadoop2.2 compilation error.

## How was this patch tested?

Existing tests.

cc tdas zsxwing

Author: Yu Peng <[email protected]>

Closes apache#15537 from loneknightpy/fix-17711.
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