RFR: 2556: LablerWorkItem adds unnecessary overhead when the bot restarts

Zhao Song zsong at openjdk.org
Wed Aug 6 21:40:52 UTC 2025


In [SKARA-2408](https://bugs.openjdk.org/browse/SKARA-2408) and [SKARA-2552](https://bugs.openjdk.org/browse/SKARA-2552), I updated the bot to do one more CheckWorkItem with force update after a pr is marked as auto labeled.

Currently, when the pr bot restarts, the bot would schedule a PullRequestCommandWorkItem for every pr(including closed prs), in the end of PullRequestCommandWorkItem, a LabelerWorkItem will be scheduled for each pr as well. In the LabelerWorkItem, when it finds the bot doesn't have label configuration, the bot will mark the pr as auto labelled and schedule a CheckWorkItem with forceUpdate. This is the issue, the bot shouldn't schedule any CheckWorkItem when the bot doesn't have label configuration. Otherwise, the bot would re-evaluate all the prs with forceUpdate.

Also, if the pr is already closed, the LabelerWorkItem shouldn't schedule any CheckWorkItem for it.

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

Commit messages:
 - update
 - update
 - SKARA-2556

Changes: https://git.openjdk.org/skara/pull/1729/files
  Webrev: https://webrevs.openjdk.org/?repo=skara&pr=1729&range=00
  Issue: https://bugs.openjdk.org/browse/SKARA-2556
  Stats: 5 lines in 1 file changed: 1 ins; 1 del; 3 mod
  Patch: https://git.openjdk.org/skara/pull/1729.diff
  Fetch: git fetch https://git.openjdk.org/skara.git pull/1729/head:pull/1729

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


More information about the skara-dev mailing list