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