tree f9abacf3ed0929632cc7ac016ba02a1d9a78f29a
parent 762423203fc1fe73ab66893b4ba3657488c1ffb4
author Edwin Kempin <ekempin@google.com> 1718622018 +0000
committer Patrick Hiesel <hiesel@google.com> 1718626343 +0000

Fix usage of ChangedFilesByPatchSetCache

This caching didn't work because we created a new instance in the
callback that is invoked per file, rather than creating one instance and
using it for the whole request.

Note, this cache is only relevant when sticky code owner approvals are
enabled.

Change-Id: Ic2357878227e5f1ffac99530f0584d42b7f1ea3b
Signed-off-by: Edwin Kempin <ekempin@google.com>
Reviewed-on: https://gerrit-review.googlesource.com/c/plugins/code-owners/+/430377
Reviewed-by: Patrick Hiesel <hiesel@google.com>
Tested-by: Zuul <zuul-63@gerritcodereview-ci.iam.gserviceaccount.com>