Skip to content

Conversation

@rxin
Copy link
Contributor

@rxin rxin commented Sep 16, 2016

What changes were proposed in this pull request?

This patch bumps the Hadoop version in hadoop-2.7 profile from 2.7.2 to 2.7.3, which was recently released and contained a number of bug fixes.

How was this patch tested?

The change should be covered by existing tests.

@rxin
Copy link
Contributor Author

rxin commented Sep 16, 2016

cc @JoshRosen @srowen

@SparkQA
Copy link

SparkQA commented Sep 16, 2016

Test build #65470 has finished for PR 15115 at commit 1fc9047.

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

@srowen
Copy link
Member

srowen commented Sep 16, 2016

+1, can't hurt

@rxin
Copy link
Contributor Author

rxin commented Sep 16, 2016

Merging in master/2.0.

asfgit pushed a commit that referenced this pull request Sep 16, 2016
## What changes were proposed in this pull request?
This patch bumps the Hadoop version in hadoop-2.7 profile from 2.7.2 to 2.7.3, which was recently released and contained a number of bug fixes.

## How was this patch tested?
The change should be covered by existing tests.

Author: Reynold Xin <rxin@databricks.com>

Closes #15115 from rxin/SPARK-17558.

(cherry picked from commit dca771b)
Signed-off-by: Reynold Xin <rxin@databricks.com>
@asfgit asfgit closed this in dca771b Sep 16, 2016
@steveloughran
Copy link
Contributor

This seems a duplicate of the #14827 patch I filed 3 weeks earlier.

Is there some aspect of the PR submission process that I'm missing out on? I would like to get my patches in, and having duplicate works wastes everyone's time.

What am I doing wrong?

@srowen
Copy link
Member

srowen commented Sep 19, 2016

The only difference was that you're also making 2.7 the default, which isn't bad or anything. Otherwise i think it was just an oversight. There's way too much traffic to keep track of unfortunately, which is why I encourage steps to reduce the noise on lists/JIRA/PRs.

You have a couple WIP PRs you've bene updating for a while so I also sort of look past yours unfortunately in the email flood as I think they're just WIP updates.

wgtmac pushed a commit to wgtmac/spark that referenced this pull request Sep 19, 2016
## What changes were proposed in this pull request?
This patch bumps the Hadoop version in hadoop-2.7 profile from 2.7.2 to 2.7.3, which was recently released and contained a number of bug fixes.

## How was this patch tested?
The change should be covered by existing tests.

Author: Reynold Xin <rxin@databricks.com>

Closes apache#15115 from rxin/SPARK-17558.
@steveloughran
Copy link
Contributor

thanks for clarifying; sometimes I feel that my patches get under reviewed —which holds for Hadoop too, where some have been outstanding for so long they're approach school age

@srowen
Copy link
Member

srowen commented Sep 20, 2016

#14731 is waiting on your response if I read it correctly; #12004 is a WIP (not sure if that's going to get merged anyway); #9571 seems to have some comments from Marcelo; #14646 I'm not sure is necessary -- was it to be committed? #13579 has a question for you from Marcelo. I'm not sure you're being ignored?

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.

4 participants