Hyrum K. Wright wrote:
> Hyrum K. Wright wrote:
>> The fs-rep-sharing branch is functionally complete, and I'd like to get the
>> branch merged to trunk soon. These are the stats for various copies of of our
>> repository for the different branch/backend combinations.
> ...
>> The remaining issue is the failing blame tests. Blame tests 10 and 11, which
>> test 'blame -g', both fail for both backends. Before the recent commits to add
>> rep-sharing to fsfs, the tests only failed for bdb. I'm slightly puzzled here
>> because 'blame -g' should be FS-agnostic. If anybody has some insight, I
>> welcome it.
>
> Just a note:
> cmpilato and I discussed the failing blame tests today, and we tracked it down
> to svn_fs_contents_changed(). Because the representations are the same,
> svn_fs_contents_changed() now returns FALSE, instead of TRUE, in cases where the
> two nodes being compared result in a empty delta. This probably impacts more
> than just 'blame -g'.
>
> It turns out that svn_fs_contents_changes() is being used to answer multiple
> questions, which until now have been answerable in the same way. We'll have to
> take a look at whether we fix the existing API, or provide another which allows
> us to differentiate these questions.
By the way, Hyrum and I decided to work around this by tracking when we
reused a rep in a way that allows for this differentiation.
--
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet <> www.collab.net <> Distributed Development On Demand
Received on 2008-10-17 04:43:56 CEST