Page MenuHomePhabricator

setdiscovery: remove unnecessary sample size limiting
ClosedPublic

Authored by martinvonz on Mar 4 2018, 11:58 AM.

Details

Summary

Both _takequicksample() and _takefullsample() already limit their
result to the request size, so there's no need to let the caller do
that again.

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

martinvonz created this revision.Mar 4 2018, 11:58 AM
indygreg accepted this revision.Mar 4 2018, 2:22 PM
This revision is now accepted and ready to land.Mar 4 2018, 2:22 PM
This revision was automatically updated to reflect the committed changes.