Page MenuHomePhabricator
Differential D19057

Completely remove the legacy hunk table
ClosedPublic
Actions

Authored by epriestley on Feb 10 2018, 8:11 PM.
Tags
None
Referenced Files
F18318923: D19057.diff
Sun, Aug 24, 12:14 PM
F18258773: D19057.diff
Fri, Aug 22, 3:14 AM
F17834232: D19057.id45690.diff
Sat, Jul 26, 2:12 PM
F17834231: D19057.id45685.diff
Sat, Jul 26, 2:11 PM
F17833725: D19057.id.diff
Sat, Jul 26, 1:48 PM
F17755974: D19057.diff
Jul 22 2025, 1:30 PM
Unknown Object (File)
May 5 2025, 6:21 PM
Unknown Object (File)
Apr 26 2025, 6:32 PM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T8475: Clean up remnants of old Differential hunk storage
T13054: Plans: Harbormaster Interactions with Differential / Chaos Reduction
Commits
rPf43d08c2bbe6: Completely remove the legacy hunk table
Summary

Depends on D19056. Fixes T8475. Ref T13054. Merges "ModernHunk" back into "Hunk".

Test Plan

Grepped for modernhunk. Reviewed revisions. Created a new revision. Used bin/differential migrate-hunk to migrate hunks between storage formats and back.

Diff Detail

Repository
rP Phabricator
Branch
hunk2
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 19487
Build 26364: Run Core Tests
Build 26363: arc lint + arc unit

Event Timeline

epriestley created this revision.Feb 10 2018, 8:11 PM
Harbormaster completed remote builds in B19487: Diff 45685.Feb 10 2018, 8:12 PM
epriestley requested review of this revision.Feb 10 2018, 8:12 PM
This revision was not accepted when it landed; it landed in state Needs Review.Feb 11 2018, 12:12 AM
Closed by commit rPf43d08c2bbe6: Completely remove the legacy hunk table (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 45685

View Options

resources/sql/autopatches/20161213.diff.01.hunks.php

Loading...
View Options

resources/sql/autopatches/20180210.hunk.02.renamemodern.sql

Loading...
View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/differential/engine/DifferentialDiffExtractionEngine.php

Loading...
View Options

src/applications/differential/management/PhabricatorDifferentialMigrateHunkWorkflow.php

Loading...
View Options

src/applications/differential/parser/__tests__/DifferentialChangesetParserTestCase.php

Loading...
View Options

src/applications/differential/parser/__tests__/DifferentialHunkParserTestCase.php

Loading...
View Options

src/applications/differential/query/DifferentialHunkQuery.php

Loading...
View Options

src/applications/differential/storage/DifferentialChangeset.php

Loading...
View Options

src/applications/differential/storage/DifferentialDiff.php

Loading...
View Options

src/applications/differential/storage/DifferentialHunk.php

Loading...
View Options

src/applications/differential/storage/DifferentialModernHunk.php

Loading...
View Options

src/applications/differential/storage/__tests__/DifferentialHunkTestCase.php

Loading...
Phabricator · Built by Phacility