Skip to content

[SPARK-14963][Minor][YARN] Fix typo in YarnShuffleService recovery file name - #14197

Closed
jerryshao wants to merge 1 commit into
apache:masterfrom
jerryshao:fix-typo-file-name
Closed

[SPARK-14963][Minor][YARN] Fix typo in YarnShuffleService recovery file name#14197
jerryshao wants to merge 1 commit into
apache:masterfrom
jerryshao:fix-typo-file-name

Conversation

@jerryshao

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Due to the changes of SPARK-14963, external shuffle recovery file name is changed mistakenly, so here change it back to the previous file name.

This only affects the master branch, branch-2.0 is correct here.

How was this patch tested?

N/A

@srowen

Copy link
Copy Markdown
Member

OK, should this be attached to SPARK-14963 then? the point being that I wouldn't want to find the original change but not this one.

@jerryshao

Copy link
Copy Markdown
ContributorAuthor

Sure, let me change the title.

@jerryshaojerryshao changed the title [Minor][YARN] Fix typo in YarnShuffleService recovery file name[SPARK-14963][Minor][YARN] Fix typo in YarnShuffleService recovery file nameJul 14, 2016
@SparkQA

Copy link
Copy Markdown

Test build #62305 has finished for PR 14197 at commit 12b7367.

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

@jerryshao

Copy link
Copy Markdown
ContributorAuthor

Ping @tgravescs , please help to review, this typo is introduce by SPARK-14963. Thanks a lot.

@tgravescs

Copy link
Copy Markdown
Contributor

+1.

Sign up for freeto 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

@jerryshao@srowen@SparkQA@tgravescs