Page MenuHomePhabricator

Update DifferentialDiff: add repositoryPHID, drop parentRevisionID
ClosedPublic

Authored by epriestley on Jan 26 2014, 5:18 PM.
Tags
None
Referenced Files
F17832852: D8072.diff
Sat, Jul 26, 1:08 PM
Unknown Object (File)
Jun 17 2025, 7:18 AM
Unknown Object (File)
Jun 15 2025, 11:59 AM
Unknown Object (File)
Jun 5 2025, 9:37 PM
Unknown Object (File)
Jun 1 2025, 5:39 AM
Unknown Object (File)
May 31 2025, 5:56 AM
Unknown Object (File)
May 30 2025, 7:16 PM
Unknown Object (File)
May 29 2025, 12:51 PM
Subscribers

Details

Summary

Moves away from ArcanistProjects:

  • Adds storage for diffs to be directly associated with a repository (instead of indirectly, through arcanist projects). Not really populated yet.
  • Drops parentRevisionID, which is obsoleted by the "Depends On" edge. This is not exposed in the UI anywhere and doesn't do anything. Resolves TODO.
Test Plan

Ran storage upgrades, browsed around, lots of grep.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley updated this revision to Unknown Object (????).Jan 26 2014, 6:42 PM
  • Also deprecate authorPHID, which isn't respected and is inauthentic.
  • Also deprecate repositoryUUID, which is redundant. Keep some traces of it around for compatibility.
  • Add UUID lookup, which is useful for SVN and can completely obsolete the older mechanism.