Skip to content

Conversation

@zsxwing
Copy link
Member

@zsxwing zsxwing commented Apr 19, 2017

What changes were proposed in this pull request?

Also went through the same file to ensure other string concatenation are correct.

How was this patch tested?

Jenkins

@SparkQA
Copy link

SparkQA commented Apr 20, 2017

Test build #75957 has finished for PR 17691 at commit a31d75b.

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

@zsxwing
Copy link
Member Author

zsxwing commented Apr 20, 2017

Thanks! Merging to master, 2.2 and 2.1.

asfgit pushed a commit that referenced this pull request Apr 20, 2017
…message

## What changes were proposed in this pull request?

Also went through the same file to ensure other string concatenation are correct.

## How was this patch tested?

Jenkins

Author: Shixiong Zhu <[email protected]>

Closes #17691 from zsxwing/fix-error-message.

(cherry picked from commit 39e303a)
Signed-off-by: Shixiong Zhu <[email protected]>
asfgit pushed a commit that referenced this pull request Apr 20, 2017
…message

## What changes were proposed in this pull request?

Also went through the same file to ensure other string concatenation are correct.

## How was this patch tested?

Jenkins

Author: Shixiong Zhu <[email protected]>

Closes #17691 from zsxwing/fix-error-message.

(cherry picked from commit 39e303a)
Signed-off-by: Shixiong Zhu <[email protected]>
@asfgit asfgit closed this in 39e303a Apr 20, 2017
@zsxwing zsxwing deleted the fix-error-message branch April 20, 2017 02:02
peter-toth pushed a commit to peter-toth/spark that referenced this pull request Oct 6, 2018
…message

## What changes were proposed in this pull request?

Also went through the same file to ensure other string concatenation are correct.

## How was this patch tested?

Jenkins

Author: Shixiong Zhu <[email protected]>

Closes apache#17691 from zsxwing/fix-error-message.
Sign up for free to 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.

3 participants