Integrated: 2563: NPE when reviewers id changed due to census update

Zhao Song zsong at openjdk.org
Fri Aug 22 18:41:48 UTC 2025


On Fri, 22 Aug 2025 17:19:11 GMT, Zhao Song <zsong at openjdk.org> wrote:

> When Skara Bot processes the backport command, it attempts to parse reviewers IDs from the original commit message. It then looks up these reviewers IDs in the current repository census to get the reviewers' full names, and then put the full names in the pull request description for backport.
> 
> A corner case may occur if the repository census is updated between the original commit and the backport creation. In this case, the reviewer ID in the original commit may be from a previous census, but later the bot could lookup the ID in the updated census. If the reviewer ID is no longer present in the updated census, this can result in a NPE.
> 
> We should fallback to use the id instead of full name.

This pull request has now been integrated.

Changeset: c79cc912
Author:    Zhao Song <zsong at openjdk.org>
URL:       https://git.openjdk.org/skara/commit/c79cc912569559241ca370ae7e774ee112409324
Stats:     9 lines in 1 file changed: 4 ins; 0 del; 5 mod

2563: NPE when reviewers id changed due to census update

Reviewed-by: erikj

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

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


More information about the skara-dev mailing list