Skip to content

[SPARK-30638][CORE][FOLLOWUP] Fix a spacing issue and use UTF-8 instead of ASCII#27420

Closed
tgravescs wants to merge 1 commit intoapache:masterfrom
tgravescs:SPARK-30638-followup
Closed

[SPARK-30638][CORE][FOLLOWUP] Fix a spacing issue and use UTF-8 instead of ASCII#27420
tgravescs wants to merge 1 commit intoapache:masterfrom
tgravescs:SPARK-30638-followup

Conversation

@tgravescs
Copy link
Contributor

What changes were proposed in this pull request?

Followup from #27367 to fix a couple of my minor issues with the Test. Fix an indentation and then use UTF-8 instead of ASCII.

Why are the changes needed?

followup

Does this PR introduce any user-facing change?

no

How was this patch tested?

compiled and ran unit test

@tgravescs tgravescs changed the title Closed [SPARK-30638][CORE][FOLLOWUP] Fix a spacing issue and use UTF-8 instead of ASCII [SPARK-30638][CORE][FOLLOWUP] Fix a spacing issue and use UTF-8 instead of ASCII Jan 31, 2020
Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, LGTM.

@SparkQA
Copy link

SparkQA commented Jan 31, 2020

Test build #117659 has finished for PR 27420 at commit 32666ee.

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

@dongjoon-hyun
Copy link
Member

Merged to master.

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.

3 participants

Comments