[remotefilelog] rfc: store refactor
Durham Goode
durham at fb.com
Mon Mar 21 00:07:05 UTC 2016
This is mainly aimed at Augie, but the refactor I did to remotefilelog to make it adhere to the ContentStore / MetadataStore pattern can be found here: https://bitbucket.org/DurhamG/remotefilelog/commits/branch/newstore (20 patches, 44d67fe through 14125f8). It's mostly code reorganization, so it's not as scary to review as it looks (there's very little new logic).
There's a bunch of naming issues (basestore.contains() is a bad name; ioutil should be moved into shallowutil; etc), but I'll probably fix that stuff as follow up patches. All the tests pass for every commit.
Augie, take a look if you get a chance. Otherwise I'll get someone at FB to review it before I push it.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.mercurial-scm.org/pipermail/mercurial-devel/attachments/20160321/edeb0d9b/attachment.html>
More information about the Mercurial-devel
mailing list