Page MenuHomePhabricator

treemanifest: move out of experimental
Needs RevisionPublic

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

Details

Reviewers
durin42
baymax
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.Oct 8 2019, 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

baymax requested changes to this revision.Jan 24 2020, 12:31 PM

There seems to have been no activities on this Diff for the past 3 Months.

By policy, we are automatically moving it out of the need-review state.

Please, move it back to need-review without hesitation if this diff should still be discussed.

:baymax:need-review-idle:

This revision now requires changes to proceed.Jan 24 2020, 12:31 PM