HomePhabricator

Increase the storage size for commit summaries

Description

Increase the storage size for commit summaries

Summary:
Fixes T11453. Currently, commit message summaries are limited to 80 bytes. This may only be 20-40 characters for CJK languages or langauges with Cyrillic script.

Increase storage size to 255, then truncate to the shorter of 255 bytes or 80 glyphs. This preserves the same behavior for latin languages, but is less tight for Russian, etc.

Some minor additional changes:

  • Provide a way to ask "how much data fits in this column?" so we don't have to duplicate column lengths across summary checks or UI errors like "title too long".
  • Remove the text80 datatype, since no other columns use it and we have no use cases (or likely use cases) for it.

Test Plan:

  • Made a commit with a Cyrillic title, saw reasonable summarization in UI:

Screen Shot 2016-08-10 at 9.46.01 AM.png (182×1 px, 29 KB)

  • Added and ran unit tests.
  • Grepped for removed SUMMARY_MAX_LENGTH constant.
  • Grepped for removed text80 data type.

Reviewers: avivey, chad

Reviewed By: avivey

Subscribers: avivey

Maniphest Tasks: T11453

Differential Revision: https://secure.phabricator.com/D16385