Skip to content

Conversation

@hauntsaninja
Copy link
Contributor

What changes were proposed in this pull request?

Fixes spacing in an error message

Why are the changes needed?

Makes error messages easier to read

Does this PR introduce any user-facing change?

Yes, it changes the error message

How was this patch tested?

This patch doesn't affect any logic, so existing tests should cover it

@HyukjinKwon
Copy link
Member

ok to test

@HyukjinKwon
Copy link
Member

Merged to master, branch-3.0 and branch-2.4.

HyukjinKwon pushed a commit that referenced this pull request Jul 28, 2020
### What changes were proposed in this pull request?
Fixes spacing in an error message

### Why are the changes needed?
Makes error messages easier to read

### Does this PR introduce _any_ user-facing change?
Yes, it changes the error message

### How was this patch tested?
This patch doesn't affect any logic, so existing tests should cover it

Closes #29264 from hauntsaninja/patch-1.

Authored-by: Shantanu <[email protected]>
Signed-off-by: HyukjinKwon <[email protected]>
(cherry picked from commit 77f2ca6)
Signed-off-by: HyukjinKwon <[email protected]>
HyukjinKwon pushed a commit that referenced this pull request Jul 28, 2020
### What changes were proposed in this pull request?
Fixes spacing in an error message

### Why are the changes needed?
Makes error messages easier to read

### Does this PR introduce _any_ user-facing change?
Yes, it changes the error message

### How was this patch tested?
This patch doesn't affect any logic, so existing tests should cover it

Closes #29264 from hauntsaninja/patch-1.

Authored-by: Shantanu <[email protected]>
Signed-off-by: HyukjinKwon <[email protected]>
(cherry picked from commit 77f2ca6)
Signed-off-by: HyukjinKwon <[email protected]>
@hauntsaninja hauntsaninja deleted the patch-1 branch July 28, 2020 02:27
@SparkQA
Copy link

SparkQA commented Jul 28, 2020

Test build #126683 has finished for PR 29264 at commit 722aa81.

  • This patch fails PySpark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

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