This is an archive of the discontinued Mercurial Phabricator instance.

revlogv2: allow upgrading to v2
ClosedPublic

Authored by Alphare on Jan 20 2021, 3:45 PM.

Details

Summary

Revlogv2 implies sidedata.
Right now sidedata is not really used in production, and Revlogv2 will be used
for the first production-ready version of sidedata support.

Diff Detail

Repository
rHG Mercurial
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Alphare created this revision.Jan 20 2021, 3:45 PM
pulkit added a subscriber: pulkit.Jan 22 2021, 11:42 AM
pulkit added inline comments.
mercurial/configitems.py
1296

A part of this seems like an unrelated change.

baymax updated this revision to Diff 25604.Feb 12 2021, 7:46 AM

โœ… refresh by Heptapod after a successful CI run (๐Ÿ™ ๐Ÿ’š)

baymax updated this revision to Diff 25616.Feb 12 2021, 6:59 PM

โœ… refresh by Heptapod after a successful CI run (๐Ÿ™ ๐Ÿ’š)

baymax updated this revision to Diff 25665.Feb 19 2021, 6:12 AM

โœ… refresh by Heptapod after a successful CI run (๐Ÿ™ ๐Ÿ’š)

baymax updated this revision to Diff 25965.Feb 25 2021, 11:45 AM

โœ… refresh by Heptapod after a successful CI run (๐Ÿ™ ๐Ÿ’š)

baymax updated this revision to Diff 26001.Mar 1 2021, 12:11 PM

โœ… refresh by Heptapod after a successful CI run (๐Ÿ™ ๐Ÿ’š)

marmoute requested changes to this revision.Mar 2 2021, 11:31 AM
marmoute added a subscriber: marmoute.
marmoute added inline comments.
mercurial/configitems.py
1295โ€“1302

This whole part seems like a mistake during a merge. We should drop it.

mercurial/upgrade_utils/actions.py
378โ€“388

if revlogv2 <=> sidedata, we should replace the associate format variant with this new one on adding a "duplicated" one.

898

revlogv1 should move to a constant for clarity/cleaness. Can you do that as preparatory changesets?

This revision now requires changes to proceed.Mar 2 2021, 11:31 AM
Alphare marked 2 inline comments as done.Mar 4 2021, 3:32 AM
Alphare updated this revision to Diff 26047.
Alphare planned changes to this revision.Mar 4 2021, 3:35 AM
Alphare added inline comments.
mercurial/upgrade_utils/actions.py
378โ€“388

I forgot to do this, my bad, I'll update

Alphare added inline comments.Mar 4 2021, 10:13 AM
mercurial/upgrade_utils/actions.py
378โ€“388

This is done in D10113

marmoute accepted this revision.Mar 9 2021, 10:52 AM
baymax updated this revision to Diff 26216.Mar 10 2021, 2:10 PM

โœ… refresh by Heptapod after a successful CI run (๐Ÿ™ ๐Ÿ’š)

This revision was not accepted when it landed; it landed in state Needs Review.
This revision was automatically updated to reflect the committed changes.