RFR: 1873: Possible notification race with tags

Zhao Song zsong at openjdk.org
Mon Apr 24 21:51:11 UTC 2023


As Erik said in the issue description, "If a new commit is integrated into a repository branch and quickly tagged, the notifier may miss updating the "Resolved In Build" field in the issue associated with that commit. This happens if the same RepositoryWorkItem instance handles both the tag and the commit, because it will handle the tag first, before a suitable backport has been created".

To solve this issue, we need to rearrange the order of handles of the tags and commits. 

I checked the logic in the two methods(**handleTags** and **handleRef**) and I couldn't find any problem would be introduced due to this change. But I could not guarantee that no problem would be found later.

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

Commit messages:
 - SKARA-1873

Changes: https://git.openjdk.org/skara/pull/1507/files
 Webrev: https://webrevs.openjdk.org/?repo=skara&pr=1507&range=00
  Issue: https://bugs.openjdk.org/browse/SKARA-1873
  Stats: 85 lines in 2 files changed: 79 ins; 4 del; 2 mod
  Patch: https://git.openjdk.org/skara/pull/1507.diff
  Fetch: git fetch https://git.openjdk.org/skara.git pull/1507/head:pull/1507

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


More information about the skara-dev mailing list