Page MenuHomePhabricator

Add database storage for a dedicated file attachment table
ClosedPublic

Authored by epriestley on May 10 2022, 11:24 PM.
Tags
None
Referenced Files
F13810785: D21813.diff
Tue, Sep 17, 7:16 PM
Unknown Object (File)
Thu, Sep 5, 10:24 AM
Unknown Object (File)
Wed, Sep 4, 11:38 PM
Unknown Object (File)
Thu, Aug 29, 7:21 AM
Unknown Object (File)
Thu, Aug 29, 2:21 AM
Unknown Object (File)
Wed, Aug 28, 11:03 AM
Unknown Object (File)
Sat, Aug 24, 9:07 PM
Unknown Object (File)
Aug 16 2024, 11:54 PM
Subscribers
Restricted Owners Package

Details

Reviewers
None
Maniphest Tasks
Restricted Maniphest Task
Commits
rPcfa42c5e655c: Add database storage for a dedicated file attachment table
Summary

Ref T13603. Prepare to move this relationship out of edge storage into dedicated storage so it is easier to formalize better in the UI.

Test Plan

Ran bin/storage upgrade.

Diff Detail

Repository
rP Phabricator
Branch
file2
Lint
Lint Passed
Unit
Test Failures
Build Status
Buildable 25710
Build 35538: arc lint + arc unit

Unit TestsFailed

TimeTest
63 msPhabricatorFileTestCase::testFileIndirectScramble
Assertion failed, expected 'true' (at PhabricatorFileTestCase.php:135): Changing attached object view policy should scramble secret. ACTUAL VALUE
227 msPhabricatorCelerityTestCase::testCelerityMaps
3 assertions passed.
9 msPhabricatorConduitTestCase::testConduitMethods
1 assertion passed.
19 msPhabricatorFileTestCase::testFileDirectScramble
2 assertions passed.
7 msPhabricatorFileTestCase::testFileStorageDelete
1 assertion passed.
View Full Test Results (1 Failed · 16 Passed)

Event Timeline

Owners added a subscriber: Restricted Owners Package.May 10 2022, 11:24 PM
This revision was not accepted when it landed; it landed in state Needs Review.May 19 2022, 8:21 PM
This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.