RFR: 1912: Show priority for bugs in pull request body [v2]

Zhao Song zsong at openjdk.org
Tue May 23 22:43:30 UTC 2023


On Tue, 23 May 2023 18:15:40 GMT, Zhao Song <zsong at openjdk.org> wrote:

>> bots/pr/src/main/java/org/openjdk/skara/bots/pr/PullRequestBot.java line 77:
>> 
>>> 75:     private final boolean enableBackport;
>>> 76:     private final Map<String, List<String>> issuePRMap;
>>> 77:     private final Map<String, Boolean> initializedPRs = new ConcurrentHashMap<>();
>> 
>> This could just be a `Set`. Also it doesn't need to be concurrent as it's local to just this PullRequestBot, and we don't run WorkItems for the same PR in parallel.
>
> Ah, right! Thanks!

Think again about this, It's possible that CheckWorkItems for different PRs in the same repo run in parallel. Every CheckWorkItem is possible to modify `initializedPRs`. So we still need synchronize it.

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

PR Review Comment: https://git.openjdk.org/skara/pull/1523#discussion_r1203110581


More information about the skara-dev mailing list