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
Unknown Object (File)
Sun, Feb 9, 8:16 AM
Unknown Object (File)
Sun, Feb 9, 8:16 AM
Unknown Object (File)
Sun, Feb 9, 8:16 AM
Unknown Object (File)
Sun, Feb 9, 8:16 AM
Unknown Object (File)
Sun, Jan 26, 10:19 PM
Unknown Object (File)
Sat, Jan 25, 5:56 AM
Unknown Object (File)
Sat, Jan 25, 5:55 AM
Unknown Object (File)
Sat, Jan 25, 5:55 AM
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