Page MenuHomePhabricator
Differential D20853

When a large number of commits need to be marked as published, issue the lookup query in chunks
ClosedPublic
Actions

Authored by epriestley on Oct 2 2019, 8:11 PM.
Tags
None
Referenced Files
F18750464: D20853.diff
Sat, Oct 4, 7:51 AM
F18745365: D20853.id.diff
Fri, Oct 3, 6:24 AM
F18734076: D20853.id.diff
Tue, Sep 30, 10:47 PM
F18678720: D20853.id49712.diff
Thu, Sep 25, 11:12 PM
F18652545: D20853.id49714.diff
Sun, Sep 21, 8:31 AM
F18570876: D20853.id49714.diff
Wed, Sep 10, 4:11 AM
F18570874: D20853.id49712.diff
Wed, Sep 10, 4:11 AM
F18567458: D20853.id.diff
Tue, Sep 9, 3:48 PM
View All Files
Subscribers
None

Details

Reviewers
None
Commits
rPed7d22142a99: (stable) When a large number of commits need to be marked as published, issue…
rP6d74736a7ec8: When a large number of commits need to be marked as published, issue the lookup…
Summary

See PHI1474. This query can become large enough to exceed reasonable packet limits. Chunk the query so it is split up if we have too many identifiers.

Test Plan

Ran bin/repository refs ... on a repository with no new commits and a repository with some new commits.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
src/
applications/
repository/
engine/
32 lines

Diff 49714

View Options

src/applications/repository/engine/PhabricatorRepositoryRefEngine.php

Loading...
Phabricator · Built by Phacility