Page MenuHomePhabricator

Allow diff generation via ArcanistBundle to be limited to an approximate maximum byte size
ClosedPublic

Authored by epriestley on May 10 2018, 5:02 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 19, 2:37 AM
Unknown Object (File)
Tue, Oct 29, 7:20 PM
Unknown Object (File)
Wed, Oct 23, 1:42 PM
Unknown Object (File)
Oct 14 2024, 6:31 AM
Unknown Object (File)
Oct 13 2024, 3:07 PM
Unknown Object (File)
Oct 9 2024, 9:36 AM
Unknown Object (File)
Sep 10 2024, 9:01 AM
Unknown Object (File)
Sep 6 2024, 8:38 AM
Subscribers
None

Details

Summary

Ref T13137. See PHI592. When you have a diff with 600MB of videos, we want to bail out of diff generation early (as soon as we realize what we're dealing with), not build an 800MB text diff in memory and then throw it away.

Support bailout during diff generation once we realize we're in over our heads.

This is approximate, but since the limit is fairly large (512KB by default) it isn't too important to be precise.

Test Plan

Rigged some callers to set various byte limits, generated diffs including diffs with large binaries. Got an appropriate diff or exeception depending on how low the limit was.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable