tree e2de7a824c85931bcde2dc9fbfb9eb542d9e2284
parent e29bcbb0b4e19e745c93814533cae8e9107bad65
author Matthias Sohn <matthias.sohn@sap.com> 1460117991 +0200
committer David Pursehouse <dpursehouse@collab.net> 1534156963 +0100

Upgrade JGit to 4.3.0.201604071810-r

this version brings the following fixes

- Fix jgit gc which retained garbage packs for 2 weeks which resulted
in repo size explosion [1] by introducing a new option for pack expiration
which by default expires packs after 1 hour

- Fix refcounting and reclaiming memory in RepositoryCache. Hugo
validated that this reduces memory consumption in Gerrit considerably

- Make the FileLfsRepository thread safe. This fixes a severe threading
issue for the lfs-storage-fs plugin [2]

Requires disabling atomic updates on InMemoryRepositoryMangers
used by acceptance tests because of this change [3] in JGit, lines
269-286. This creates an anonymous subclass of BatchRefUpdate and when
running Gerrit's tests the code always enters the if-branch which
doesn't update the progress monitor. Since the progress monitor is not
updated the "refs: 1" is missing from the output and the assertion in
the line 469 fails.

When running normal (not from tests) Gerrit daemon this JGit code is
not used and the issue doesn't exist.

This is an intermediate step to upgrading to 4.5.4.201711221230-r which
keeps the branch building and tests passing.

Atomic update bug description re-worded from Saša Živkov's comment on [4].

[1] https://groups.google.com/forum/#!searchin/repo-discuss/gc/repo-discuss/JW06GsH0vLo/9GV4TL0VDgAJ
[2] https://git.eclipse.org/r/#/c/52177/16/org.eclipse.jgit.lfs.server/src/org/eclipse/jgit/lfs/server/fs/FileLfsServlet.java
[3] https://git.eclipse.org/r/#/c/61841/2/org.eclipse.jgit/src/org/eclipse/jgit/internal/storage/dfs/InMemoryRepository.java
[4] https://gerrit-review.googlesource.com/c/gerrit/+/77110/1/gerrit-acceptance-tests/src/test/java/com/google/gerrit/acceptance/git/AbstractPushForReview.java#b496

Change-Id: If6ba90c5e79747d20338ad32f03624a6bd75ee43
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>