Page MenuHomePhabricator

Make "metamta.differential.inline-patches" imply a reasonable byte limit, not just a line limit
ClosedPublic

Authored by epriestley on Oct 17 2016, 10:30 PM.
Tags
None
Referenced Files
F14089042: D16714.diff
Sun, Nov 24, 7:41 AM
Unknown Object (File)
Tue, Nov 19, 12:00 PM
Unknown Object (File)
Tue, Oct 29, 10:19 PM
Unknown Object (File)
Oct 22 2024, 8:40 AM
Unknown Object (File)
Oct 20 2024, 6:02 AM
Unknown Object (File)
Oct 18 2024, 1:54 AM
Unknown Object (File)
Oct 9 2024, 5:29 AM
Unknown Object (File)
Oct 9 2024, 5:29 AM
Subscribers
None

Details

Summary

Fixes T11748. This option currently implies a line limit (e.g., inline patches that are less than 100 lines long). This breaks down if a diff has a 10MB line, like a huge blob of JSON all on one line.

For now, imply a reasonable byte limit (256 bytes per line).

See T11767 for future work to make this and related options more cohesive.

Test Plan
  • With option at 1000: sent Differential email, saw patches inlined.
  • With option at 10: sent Differential email, saw patches dropped because of the byte limit.
  • var_dump()'d the actual limits and used bin/worker execute --id ... to sanity check that things were working properly.

Diff Detail

Repository
rP Phabricator
Branch
mbytes1
Lint
Lint Passed
SeverityLocationCodeMessage
Advicesrc/applications/differential/editor/DifferentialTransactionEditor.php:1288XHP16TODO Comment
Unit
Tests Passed
Build Status
Buildable 14142
Build 18367: Run Core Tests
Build 18366: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to Make "metamta.differential.inline-patches" imply a reasonable byte limit, not just a line limit.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Oct 17 2016, 10:55 PM
This revision was automatically updated to reflect the committed changes.