This is an archive of the discontinued Mercurial Phabricator instance.

hgweb: extract code for emitting multiple changelist records
ClosedPublic

Authored by indygreg on May 17 2018, 3:27 PM.

Details

Summary

changelistentry() exists so extensions can easily monkeypatch the
function to add additional metadata.

In at least one case at Mozilla, we have an extension doing this
where performance would greatly benefit if we were able to "batch
fetch" metadata for all revisions that will eventually have their
data set.

By extracting the logic for "obtain records for N revisions" into
a standalone function, we make it possible for a monkeypatched
function to more efficiently bulk fetch data for N revisions.

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

indygreg created this revision.May 17 2018, 3:27 PM
pulkit accepted this revision.May 18 2018, 9:52 AM
This revision was automatically updated to reflect the committed changes.