Page MenuHomePhabricator

treemanifest: move out of experimental
Needs ReviewPublic

Authored by pulkit on Tue, Oct 8, 5:03 PM.

Details

Reviewers
durin42
Group Reviewers
hg-reviewers
Summary

In recent sprint, we discussed that treemanifest should be moved out of
experimental. The feature has been used inside Google, Facebook in some forms
and was used inside Yandex too for narrow clones.

There is an issue with diffing of large treemanifests and to fix that some
Python code needs to be rewritted in C/Rust. This I think is not a blocker for
taking treemanifest out of experimental.

This patch renames config experimental.treemanifest to storage.treemanifest.

Diff Detail

Repository
rHG Mercurial
Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

pulkit created this revision.Tue, Oct 8, 5:03 PM

the storage session is more about small configuration of the storage
system without impact on the repository. The format section is more
appropriate for things that deeply impact the repository format.

So this should probably at least lives in the format as
format.use-tree-manifest.

However, this is also the first "scheme" change we get. repository using
tree-manifest will never be able to push to a repository not using it.
That make tree-manifest different from anything that live in the
format section right now.

So maybe: format.manifest-scheme=tree-manifest ?

That sounds fine