Skip to content

Conversation

@mdianjun
Copy link

What changes were proposed in this pull request?

Correct a word in a log message.

Why are the changes needed?

Log message will be more clearly.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Test is not needed.

@maropu maropu changed the title Correct a log message [CORE][MINOR] Correct a log message in DAGScheduler Sep 21, 2019
@maropu
Copy link
Member

maropu commented Sep 21, 2019

ok to test

@SparkQA
Copy link

SparkQA commented Sep 21, 2019

Test build #111112 has finished for PR 25880 at commit 814cfad.

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

@HyukjinKwon
Copy link
Member

It's fine but can you check if there are some other typos too while we're here?

@mdianjun
Copy link
Author

It's fine but can you check if there are some other typos too while we're here?

Yes, I have checked the whole project.

@dongjoon-hyun
Copy link
Member

Thank you for making PR, @mdianjun .

@SparkQA
Copy link

SparkQA commented Sep 22, 2019

Test build #111137 has finished for PR 25880 at commit 640b2c8.

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

@HyukjinKwon
Copy link
Member

Merged to master and branch-2.4.

HyukjinKwon pushed a commit that referenced this pull request Sep 22, 2019
### What changes were proposed in this pull request?

Correct a word in a log message.

### Why are the changes needed?

Log message will be more clearly.

### Does this PR introduce any user-facing change?

No.

### How was this patch tested?

Test is not needed.

Closes #25880 from mdianjun/fix-a-word.

Authored-by: madianjun <[email protected]>
Signed-off-by: HyukjinKwon <[email protected]>
(cherry picked from commit e2c4787)
Signed-off-by: HyukjinKwon <[email protected]>
@mdianjun mdianjun deleted the fix-a-word branch September 22, 2019 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants