Page MenuHomePhabricator

infinitepush: add reponame to WHERE clause
ClosedPublic

Authored by stash on Oct 27 2017, 5:13 AM.

Details

Reviewers
durham
Group Reviewers
Restricted Project
Commits
rFBHGXdd3eb0168d25: infinitepush: add reponame to WHERE clause
Summary

Since reponame is part of the primary key, it also makes sense to add it to
WHERE clause. Otherwise query may trigger full table scan and it can be very
slow.

Test Plan

Run unittest

Diff Detail

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

Event Timeline

stash created this revision.Oct 27 2017, 5:13 AM
Herald added a reviewer: Restricted Project. · View Herald TranscriptOct 27 2017, 5:13 AM
durham accepted this revision.Oct 31 2017, 1:01 PM
This revision is now accepted and ready to land.Oct 31 2017, 1:01 PM
This revision was automatically updated to reflect the committed changes.