Skip to content

Conversation

@jiangxb1987
Copy link
Contributor

@jiangxb1987 jiangxb1987 commented Nov 6, 2017

Closes #11494
Closes #14158
Closes #16803
Closes #16864
Closes #17455
Closes #17936
Closes #19377

Added:
Closes #19380
Closes #18642
Closes #18377
Closes #19632

Added:
Closes #14471
Closes #17402
Closes #17953
Closes #18607

Also cc @srowen @vanzin @HyukjinKwon @gatorsmile @cloud-fan to see if you have other PRs to close.

@srowen
Copy link
Member

srowen commented Nov 6, 2017

I would close:

#19380
#18642
#18377
#19632

@robert3005
Copy link

#18406 isn't stale, thanks

@HyukjinKwon
Copy link
Member

Mine is:

#14471
#17402
#17953
#18607

rest of them are basically a subset of the current list.

@jiangxb1987
Copy link
Contributor Author

Removed #18406

@SparkQA
Copy link

SparkQA commented Nov 6, 2017

Test build #83484 has finished for PR 19669 at commit 33c60fa.

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

Copy link
Member

@gatorsmile gatorsmile left a comment

Choose a reason for hiding this comment

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

LGTM

@HyukjinKwon
Copy link
Member

Merged to master.

@asfgit asfgit closed this in ed1478c Nov 7, 2017
@jiangxb1987 jiangxb1987 deleted the stale-prs branch November 7, 2017 11:20
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.

7 participants