subversion-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Philip Martin <philip.mar...@wandisco.com>
Subject Re: svn commit: r1716548 - /subversion/trunk/subversion/tests/libsvn_fs/fs-test.c
Date Fri, 27 Nov 2015 09:27:40 GMT
Daniel Shahaf <d.s@daniel.shahaf.name> writes:

> Philip Martin wrote on Thu, Nov 26, 2015 at 13:46:39 +0000:
>> I suppose one way to fix this would be to ensure that every BDB revision
>> generates a new node-revision-id.
>
> I wouldn't call this a fix; I think it is a workaround.  A "fix" would
> be to figure out why bdb reports the wrong revision number, not to make
> the place it wrongly looks the value up in contain the right value.

I don't understand whay you want.  BDB does not store the base revision
in the txn, it stores the base node-revision-id instead.  Revisions
usually map 1:1 to the base node-revision-ids but the editor drive
"begin_txn, commit" breaks this because it creates a revision that
shares the base node-revision-id with the previous revision.  My patch
makes the mapping 1:1 so storing the node-revision-id works.

If you expect a patch that causes BDB to change the txn storage on disk
to include the base revision (or instead of the base node-revision-id?)
then you will have to explain why that is better.

-- 
Philip Martin
WANdisco

Mime
View raw message