diff options
author | Luca Milanesio <luca.milanesio@gmail.com> | 2022-10-28 21:24:36 +0100 |
---|---|---|
committer | Luca Milanesio <luca.milanesio@gmail.com> | 2022-10-28 21:24:50 +0100 |
commit | 9a0822b613f346ce43eada6a893e7121201270dd (patch) | |
tree | ee450d1d4a35ae6f34d14c939c805b46e838469d | |
parent | b20005626546964729e341465f835daa91141916 (diff) | |
parent | b5e7cedd00abd8e48ec670bbe845b4f009912974 (diff) |
Merge branch 'stable-2.16' into stable-3.0
* stable-2.16:
Cache repository locations in LocalDiskRepositoryManager
Release-Notes: skip
Change-Id: I39713079d081e7b80f59dc36e6181f4daa565c50
-rw-r--r-- | java/com/google/gerrit/server/git/LocalDiskRepositoryManager.java | 21 |
1 files changed, 15 insertions, 6 deletions
diff --git a/java/com/google/gerrit/server/git/LocalDiskRepositoryManager.java b/java/com/google/gerrit/server/git/LocalDiskRepositoryManager.java index 1fdf194016..2c0fdca83f 100644 --- a/java/com/google/gerrit/server/git/LocalDiskRepositoryManager.java +++ b/java/com/google/gerrit/server/git/LocalDiskRepositoryManager.java @@ -33,8 +33,10 @@ import java.nio.file.SimpleFileVisitor; import java.nio.file.attribute.BasicFileAttributes; import java.util.Collections; import java.util.EnumSet; +import java.util.Map; import java.util.SortedSet; import java.util.TreeSet; +import java.util.concurrent.ConcurrentHashMap; import org.eclipse.jgit.errors.RepositoryNotFoundException; import org.eclipse.jgit.lib.Config; import org.eclipse.jgit.lib.ConfigConstants; @@ -108,6 +110,7 @@ public class LocalDiskRepositoryManager implements GitRepositoryManager { } private final Path basePath; + private final Map<Project.NameKey, FileKey> fileKeyByProject = new ConcurrentHashMap<>(); @Inject LocalDiskRepositoryManager(SitePaths site, @GerritServerConfig Config cfg) { @@ -129,17 +132,23 @@ public class LocalDiskRepositoryManager implements GitRepositoryManager { @Override public Repository openRepository(Project.NameKey name) throws RepositoryNotFoundException { - return openRepository(getBasePath(name), name); - } + FileKey cachedLocation = fileKeyByProject.get(name); + if (cachedLocation != null) { + try { + return RepositoryCache.open(cachedLocation); + } catch (IOException e) { + fileKeyByProject.remove(name, cachedLocation); + } + } - private Repository openRepository(Path path, Project.NameKey name) - throws RepositoryNotFoundException { if (isUnreasonableName(name)) { throw new RepositoryNotFoundException("Invalid name: " + name); } - FileKey loc = FileKey.lenient(path.resolve(name.get()).toFile(), FS.DETECTED); + FileKey location = FileKey.lenient(getBasePath(name).resolve(name.get()).toFile(), FS.DETECTED); try { - return RepositoryCache.open(loc); + Repository repo = RepositoryCache.open(location); + fileKeyByProject.put(name, location); + return repo; } catch (IOException e) { throw new RepositoryNotFoundException("Cannot open repository " + name, e); } |