Page MenuHomePhabricator

Use a hash-and-diff strategy to produce a diff layout for block-based documents
ClosedPublic

Authored by epriestley on Sep 25 2019, 9:05 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 17, 2:54 PM
Unknown Object (File)
Tue, Apr 9, 11:45 AM
Unknown Object (File)
Sat, Apr 6, 8:53 AM
Unknown Object (File)
Mon, Apr 1, 7:15 AM
Unknown Object (File)
Sat, Mar 30, 9:16 AM
Unknown Object (File)
Thu, Mar 28, 11:28 AM
Unknown Object (File)
Mar 20 2024, 8:30 AM
Unknown Object (File)
Mar 16 2024, 2:22 AM
Subscribers
None

Details

Summary

Depends on D20835. Ref T13425. Ref T13414. When a document has a list of content blocks, we may not be able to diff it directly, but we can hash each block and then diff the hashes (internally "diff" also does approximately the same thing).

We could do this ourselves with slightly fewer layers of indirection, but: diff already exists; we already use it; we already have a bunch of abstractions on top of it; and it's likely much faster on large inputs than the best we can do in PHP.

Test Plan

Screen Shot 2019-09-25 at 1.57.05 PM.png (892×1 px, 149 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable