D6732: localrepo: push manifestlog and changelog construction code into store

indygreg (Gregory Szorc) phabricator at mercurial-scm.org
Wed Aug 21 11:46:03 EDT 2019


This revision is now accepted and ready to land.
indygreg added a comment.
indygreg accepted this revision.


  The boundary between `localrepository` and `store` is a bit murky and definitely needs some work. When I last had things paged into my brain last year, I think I was trending towards breaking up the store into discrete classes and making the code that dynamically resolves repository types cherry pick the classes depending on the opener options. But we could easily have the store handle this logic as well. This is all very complicated and I don't prescribe to have a single correct answer.

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST ACTION
  https://phab.mercurial-scm.org/D6732/new/

REVISION DETAIL
  https://phab.mercurial-scm.org/D6732

To: durin42, #hg-reviewers, indygreg
Cc: indygreg, mercurial-devel


More information about the Mercurial-devel mailing list