RFR: 1629: Limit how far back maxUpdatedAt can be set in PullRequestPoller [v2]

[Erik Helin]

On Thu, 13 Oct 2022 14:07:07 GMT, Erik Joelsson <erikj at openjdk.org> wrote:

>> After having observed the PullRequestPoller in action for a bit, I've noticed some weird behavior. Say a very old open PR A exists, and there is a closed PR B that is newer (but older than the 7 day initial query limit). Then on the first round, PR A will be returned and maxUpdatedAt will be set to updatedAt of PR A. On the second round, this will cause PR B to be returned. This is causing some weird behavior, resurrecting very old closed PRs that have problems with them.
>> 
>> This patch fixes this by putting a hard limit on how far back we query for the updatedAt field.
>
> Erik Joelsson has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Recalculate limit dynamically

Looks even better 😉

-------------

Marked as reviewed by ehelin (Reviewer).

PR: https://git.openjdk.org/skara/pull/1395