Do not store ObjectId subclasses in RefDatabase caches
commit4af1edad36e729ee846ba5e943949b19fc39e744
authorShawn O. Pearce <spearce@spearce.org>
Wed, 18 Feb 2009 03:36:51 +0000 (17 19:36 -0800)
committerRobin Rosenberg <robin.rosenberg@dewire.com>
Wed, 18 Feb 2009 06:41:11 +0000 (18 07:41 +0100)
treed6c41ee9e50cc4801e8fa157bac5210fd2d36af3
parentfedd114bcb8ef859b9cc2152bf136a554dc54946
Do not store ObjectId subclasses in RefDatabase caches

We cannot permit an ObjectId subclass, such as RevCommit, to be
cached inside of the RefDatabase cache.  Instead of toObjectId()
use copy() to ensure we get exactly an ObjectId for the new value,
as that is the instance we will cache in the RefDatabase.

Caching the ObjectId subclass in RefDatabase will cause any later
cached read of the ref to return that RevCommit, confusing all
application code using "ref.getObjectId().equals(...)" due to the
overridden definition of equals(ObjectId).

ReceivePack was bitten by this, sometimes reporting "invalid old id"
to clients if the ref was last updated using a RevCommit instead
of an ObjectId.  It also caused the history of that repository to
be pinned in memory as RevCommit instances, all of them reachable
from the RefDatabase cache of the last updated Ref.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
Signed-off-by: Robin Rosenberg <robin.rosenberg@dewire.com>
org.spearce.jgit.test/tst/org/spearce/jgit/lib/RefUpdateTest.java
org.spearce.jgit/src/org/spearce/jgit/lib/RefUpdate.java