( )⚙ D4583 shallowbundle: dump this revision prefetching logic

This is an archive of the discontinued Mercurial Phabricator instance.

shallowbundle: dump this revision prefetching logic
ClosedPublic

Authored by durin42 on Sep 13 2018, 6:09 PM.
Tags
None
Subscribers

Details

Reviewers
quark
Group Reviewers
Restricted Project
Commits
rFBHGX12c6ed203897: shallowbundle: dump this revision prefetching logic
Summary

junw and I talked about this and we're both reasonably convinced this
is defending against a rare-in-practice case that could only result in
slowness, rather than incorrectness: missing cached blobs of
already-public changes.

Done as a separate follow-up change to give us something easy to back
out if it turns out we were wrong. :)

Diff Detail

Repository
rFBHGX Facebook Mercurial Extensions
Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

durin42 created this revision.Sep 13 2018, 6:09 PM
Herald added a reviewer: Restricted Project. · View Herald TranscriptSep 13 2018, 6:09 PM
quark accepted this revision.Sep 13 2018, 6:12 PM
quark added a subscriber: quark.

Pushed. Thanks!

This revision is now accepted and ready to land.Sep 13 2018, 6:12 PM
This revision was automatically updated to reflect the committed changes.