Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Check stale issues in ascending order #9489

Merged
merged 1 commit into from
Jan 17, 2024
Merged

Conversation

manuzhang
Copy link
Contributor

We have so many issues that only latest ones are checked for staleness due to rate limit.

We have so many issues that only latest ones are checked for staleness due to rate limit
@github-actions github-actions bot added the INFRA label Jan 17, 2024
@@ -47,3 +47,4 @@ jobs:
close-issue-message: >
This issue has been closed because it has not received any activity in the last 14 days
since being marked as 'stale'
ascending: true
Copy link
Member

Choose a reason for hiding this comment

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

It does make sense to close the older stale issues first. But I was wondering was there any default limit to number of issues to be processed? Why it is not closing all the issues that are older than 180 days and stale for 14 days.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

the workflow triggers daily at midnight, considering 30 issues a day. It should have closed by now?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

With the default descending order, it always checks the latest issues and only a small amount can be closed. Older issues are never checked.

Copy link
Contributor

@nastra nastra left a comment

Choose a reason for hiding this comment

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

LGTM, let's see if that makes a difference and old issues get closed

@nastra nastra merged commit b327327 into apache:main Jan 17, 2024
41 checks passed
geruh pushed a commit to geruh/iceberg that referenced this pull request Jan 26, 2024
We have so many issues that only latest ones are checked for staleness due to rate limit
adnanhemani pushed a commit to adnanhemani/iceberg that referenced this pull request Jan 30, 2024
We have so many issues that only latest ones are checked for staleness due to rate limit
devangjhabakh pushed a commit to cdouglas/iceberg that referenced this pull request Apr 22, 2024
We have so many issues that only latest ones are checked for staleness due to rate limit
@manuzhang manuzhang deleted the patch-2 branch June 4, 2024 11:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants