Repository navigation
pr-is-up-to-date: Speed up processing of tag pushes with paths - #23123
Conversation
When processing a tag push and PATHS is set, we need to find the merge base to get the diff to determine if the paths are touched. But it's a lot faster to fetch 100 revisions at once than to fetch 10 revisions 10 times. So instead of fetching to each PR's merge base, which will likely enough fetch a few revisions for every PR beyond a certain age, fetch arbitrary blocks of revisions.
|
Thank you for your PR! When contributing to Jetpack, we have a few suggestions that can help us test and review your patch:
This comment will be updated as you work on your PR and make changes. If you think that some of those checks are not needed for your PR, please explain why you think so. Thanks for cooperation 🤖 The e2e test report can be found here. Please note that it can take a few minutes after the e2e tests checks are complete for the report to be available. Once your PR is ready for review, check one last time that all required checks (other than "Required review") appearing at the bottom of this PR are passing or skipped. |
Changes proposed in this Pull Request:
When processing a tag push and PATHS is set, we need to find the merge
base to get the diff to determine if the paths are touched.
But it's a lot faster to fetch 100 revisions at once than to fetch 10
revisions 10 times. So instead of fetching to each PR's merge base,
which will likely enough fetch a few revisions for every PR beyond a
certain age, fetch arbitrary blocks of revisions.
Jetpack product discussion
None
Does this pull request change what data or activity we track or use?
No
Testing instructions: