Skip to content

[BUILD] Close stale PRs - #17734

Closed
maropu wants to merge 1 commit into
apache:masterfrom
maropu:resolved_pr
Closed

[BUILD] Close stale PRs#17734
maropu wants to merge 1 commit into
apache:masterfrom
maropu:resolved_pr

Conversation

@maropu

@maropumaropu commented Apr 23, 2017

Copy link
Copy Markdown
Member

What changes were proposed in this pull request?

This pr proposed to close stale PRs. Currently, we have 400+ open PRs and there are some stale PRs whose JIRA tickets have been already closed and whose JIRA tickets does not exist (also, they seem not to be minor issues).

// Open PRs whose JIRA tickets have been already closed
Closes#11785
Closes#13027
Closes#13614
Closes#13761
Closes#15197
Closes#14006
Closes#12576
Closes#15447
Closes#13259
Closes#15616
Closes#14473
Closes#16638
Closes#16146
Closes#17269
Closes#17313
Closes#17418
Closes#17485
Closes#17551
Closes#17463
Closes#17625

// Open PRs whose JIRA tickets does not exist and they are not minor issues
Closes#10739
Closes#15193
Closes#15344
Closes#14804
Closes#16993
Closes#17040
Closes#15180
Closes#17238

How was this patch tested?

N/A

@maropu

maropu commented Apr 23, 2017

Copy link
Copy Markdown
MemberAuthor

IIUC I listed up the PRs that it is worth closing. Cloud you check this? (I pinged you cuz you opened possibly similar PRs in the past) cc: @srowen@HyukjinKwon

@HyukjinKwonHyukjinKwon left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I hope the PRs in the latter case are double-checked by committers while this PR is open though.

@maropu

Copy link
Copy Markdown
MemberAuthor

Thanks for checking, guys! yea, I also think the double-checking is needed there.

@HyukjinKwon

Copy link
Copy Markdown
Member

Oh @maropu, BTW, does just "Close" work too? I haven't tested the merge script but I thought it should be "Closes".

@maropu

Copy link
Copy Markdown
MemberAuthor

oh, yea, I'll fix soon. Thanks!

@SparkQA

Copy link
Copy Markdown

Test build #76079 has finished for PR 17734 at commit e4c2e7d.

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

@SparkQA

Copy link
Copy Markdown

Test build #76082 has finished for PR 17734 at commit 559854f.

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

@maropu
maropu deleted the resolved_pr branch July 5, 2017 11:51
peter-toth pushed a commit to peter-toth/spark that referenced this pull request Oct 6, 2018
This pr proposed to close stale PRs. Currently, we have 400+ open PRs and there are some stale PRs whose JIRA tickets have been already closed and whose JIRA tickets does not exist (also, they seem not to be minor issues).
// Open PRs whose JIRA tickets have been already closed
Closesapache#11785Closesapache#13027Closesapache#13614Closesapache#13761Closesapache#15197Closesapache#14006Closesapache#12576Closesapache#15447Closesapache#13259Closesapache#15616Closesapache#14473Closesapache#16638Closesapache#16146Closesapache#17269Closesapache#17313Closesapache#17418Closesapache#17485Closesapache#17551Closesapache#17463Closesapache#17625
// Open PRs whose JIRA tickets does not exist and they are not minor issues
Closesapache#10739Closesapache#15193Closesapache#15344Closesapache#14804Closesapache#16993Closesapache#17040Closesapache#15180Closesapache#17238
N/A
Author: Takeshi Yamamuro <yamamuro@apache.org>
Closesapache#17734 from maropu/resolved_pr.
Change-Id: Id2e590aa7283fe5ac01424d30a40df06da6098b5
zifeif2 pushed a commit to zifeif2/spark that referenced this pull request Nov 22, 2025
## What changes were proposed in this pull request?
This pr proposed to close stale PRs. Currently, we have 400+ open PRs and there are some stale PRs whose JIRA tickets have been already closed and whose JIRA tickets does not exist (also, they seem not to be minor issues).
// Open PRs whose JIRA tickets have been already closed
Closesapache#11785Closesapache#13027Closesapache#13614Closesapache#13761Closesapache#15197Closesapache#14006Closesapache#12576Closesapache#15447Closesapache#13259Closesapache#15616Closesapache#14473Closesapache#16638Closesapache#16146Closesapache#17269Closesapache#17313Closesapache#17418Closesapache#17485Closesapache#17551Closesapache#17463Closesapache#17625
// Open PRs whose JIRA tickets does not exist and they are not minor issues
Closesapache#10739Closesapache#15193Closesapache#15344Closesapache#14804Closesapache#16993Closesapache#17040Closesapache#15180Closesapache#17238
## How was this patch tested?
N/A
Author: Takeshi Yamamuro <yamamuro@apache.org>
Closesapache#17734 from maropu/resolved_pr.
Sign up for freeto 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.

4 participants

@maropu@HyukjinKwon@SparkQA@srowen