Skip to content

[MINOR] Fix a typo in FsHistoryProvider loginfo#28505

Closed
sungpeo wants to merge 1 commit intoapache:masterfrom
sungpeo:typo_fshistoryprovider
Closed

[MINOR] Fix a typo in FsHistoryProvider loginfo#28505
sungpeo wants to merge 1 commit intoapache:masterfrom
sungpeo:typo_fshistoryprovider

Conversation

@sungpeo
Copy link

@sungpeo sungpeo commented May 12, 2020

What changes were proposed in this pull request?

a typo in logging. (just added : )

@HyukjinKwon
Copy link
Member

It's okay but mind double checking if there are some more typos around here? I am sure there are more and let's fix it while we're here.

@HyukjinKwon
Copy link
Member

@HyukjinKwon HyukjinKwon changed the title Fix a typo in FsHistoryProvider loginfo [MINOR] Fix a typo in FsHistoryProvider loginfo May 12, 2020
@srowen
Copy link
Member

srowen commented May 14, 2020

Jenkins test this please

@SparkQA
Copy link

SparkQA commented May 14, 2020

Test build #122620 has finished for PR 28505 at commit 679a1bc.

  • This patch fails build dependency tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented May 14, 2020

Test build #5004 has finished for PR 28505 at commit 679a1bc.

  • This patch fails build dependency tests.
  • This patch does not merge cleanly.
  • This patch adds no public classes.

@srowen
Copy link
Member

srowen commented May 16, 2020

Jenkins test this please

@SparkQA
Copy link

SparkQA commented May 16, 2020

Test build #122729 has finished for PR 28505 at commit 679a1bc.

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

@srowen srowen closed this in 097d509 May 17, 2020
@srowen
Copy link
Member

srowen commented May 17, 2020

OK, merged to master

@sungpeo sungpeo deleted the typo_fshistoryprovider branch May 17, 2020 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants