D5308: store: don't pass 'atomictemp=True' while appending to fncache
yuja (Yuya Nishihara)
phabricator at mercurial-scm.org
Tue Nov 27 08:37:48 EST 2018
yuja added a comment.
> - a/mercurial/store.py +++ b/mercurial/store.py @@ -486,7 +486,7 @@ if self.addls:
> 1. if we have just new entries, let's append them to the fncache tr.addbackup('fncache')
> - fp = self.vfs('fncache', mode='ab', atomictemp=True) + fp = self.vfs('fncache', mode='ab')
Ah, no. addbackup() creates hardlink, which means fncache can't be updated in
place. Also, the reader wouldn't handle partially-written fncache well.
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D5308
To: pulkit, #hg-reviewers
Cc: yuja, mjpieters, mercurial-devel
More information about the Mercurial-devel
mailing list