Skip to content

Conversation

@HyukjinKwon
Copy link
Member

@HyukjinKwon HyukjinKwon commented Jul 16, 2018

Closes #17422
Closes #17619
Closes #18034
Closes #18229
Closes #18268
Closes #17973
Closes #18125
Closes #18918
Closes #19274
Closes #19456
Closes #19510
Closes #19420
Closes #20090
Closes #20177
Closes #20304
Closes #20319
Closes #20543
Closes #20437
Closes #21261
Closes #21726
Closes #14653
Closes #13143
Closes #17894
Closes #19758
Closes #12951
Closes #17092
Closes #21240
Closes #16910
Closes #12904
Closes #21731
Closes #21095

Added:
Closes #19233
Closes #20100
Closes #21453
Closes #21455
Closes #18477

Added:
Closes #21812
Closes #21787

@HyukjinKwon
Copy link
Member Author

HyukjinKwon commented Jul 16, 2018

Mostly they are proposed to be closed by

  1. author's inactivity
  2. committer's decision to close (including me)
  3. account removed
  4. PRs that are being taken over
  5. etc.

Authors:

  • Please come and say "please take out #XXXXX". I will leave this PR open for few days.
  • If you guys missed it, please reopen when you guys find some time. That would give more refreshes to make the progress on the proposed changes.

Contributors:
Please go ahead if you guys find some PRs worth taking over. Please leave a comment and open a PR.

@SparkQA
Copy link

SparkQA commented Jul 16, 2018

Test build #93080 has finished for PR 21781 at commit 9f7e2a2.

  • This patch fails to generate documentation.
  • This patch merges cleanly.
  • This patch adds no public classes.

@jiangxb1987
Copy link
Contributor

I checked the listed PRs in Core module and it seems fine to close them. Also cc @gatorsmile

@10110346
Copy link
Contributor

please take out #21079

@srowen
Copy link
Member

srowen commented Jul 18, 2018

I would add:

#19233
#20100
#21453
#21455
#18477

@HyukjinKwon
Copy link
Member Author

Merged to master.

@asfgit asfgit closed this in 1a4fda8 Jul 19, 2018
@HyukjinKwon HyukjinKwon deleted the closing-prs branch October 16, 2018 12:46
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