Skip to content

Conversation

@srowen
Copy link
Member

@srowen srowen commented Oct 25, 2018

What changes were proposed in this pull request?

Add note about AddJar return value change in migration guide

How was this patch tested?

n/a

@SparkQA
Copy link

SparkQA commented Oct 25, 2018

Test build #98024 has finished for PR 22826 at commit 419414c.

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

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.

Thank you, @srowen !
cc @gatorsmile

Copy link
Member

@HyukjinKwon HyukjinKwon left a comment

Choose a reason for hiding this comment

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

Looks okay

@srowen
Copy link
Member Author

srowen commented Oct 26, 2018

Merged to master

@asfgit asfgit closed this in f1891ff Oct 26, 2018
@srowen srowen deleted the SPARK-25760.2 branch October 27, 2018 19:23
jackylee-ch pushed a commit to jackylee-ch/spark that referenced this pull request Feb 18, 2019
…ge in migration guide

## What changes were proposed in this pull request?

Add note about AddJar return value change in migration guide

## How was this patch tested?

n/a

Closes apache#22826 from srowen/SPARK-25760.2.

Authored-by: Sean Owen <[email protected]>
Signed-off-by: Sean Owen <[email protected]>
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.

5 participants