Page MenuHomePhabricator

Skip loading transaction handles in an old migration
ClosedPublic

Authored by epriestley on Jun 5 2019, 6:43 PM.
Tags
None
Referenced Files
F18820025: D20570.id.diff
Wed, Oct 22, 12:03 PM
F18817189: D20570.diff
Tue, Oct 21, 1:18 PM
F18733976: D20570.id.diff
Tue, Sep 30, 10:45 PM
F18705557: D20570.diff
Sun, Sep 28, 11:56 AM
F18705418: D20570.diff
Sun, Sep 28, 11:17 AM
F18657394: D20570.id49073.diff
Sep 23 2025, 1:37 AM
F18636889: D20570.id.diff
Sep 17 2025, 5:26 AM
F18628036: D20570.id.diff
Sep 16 2025, 5:25 AM
Subscribers
Restricted Owners Package

Details

Summary

Ref T13305. See that task for discussion.

This old migration may indirectly cause search index worker tasks to queue by loading handles. They'll fail since we later added dateCreated to the worker task table.

Use needHandles(false) (since we don't use them) to disable loading handles and avoid the problem.

Test Plan
  • Ran bin/storage upgrade -f on an older instance (late 2016) and hit this issue.
  • Applied the patch, got a clean migration to modernity.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable