Page MenuHomePhabricator

Survive importing Git commits with no commit message and/or no author
ClosedPublic

Authored by epriestley on May 19 2020, 3:00 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Nov 21, 9:12 AM
Unknown Object (File)
Tue, Nov 19, 10:04 AM
Unknown Object (File)
Wed, Nov 13, 12:36 AM
Unknown Object (File)
Mon, Nov 4, 10:47 AM
Unknown Object (File)
Mon, Nov 4, 10:46 AM
Unknown Object (File)
Mon, Nov 4, 10:46 AM
Unknown Object (File)
Sun, Nov 3, 2:00 AM
Unknown Object (File)
Oct 19 2024, 2:15 PM
Subscribers
None

Details

Summary

Ref T13538. See PHI1739. Synthetic Git commits with no author and/or no commit message currently extract null and then fail to parse.

Ideally, we would carefully distinguish between null and empty string. In practice, that requires significant schema changes (these columns are non-nullable and have indexing requirements) and these cases are degenerate. These commits are challenging to build and can not normally be constructed with git commit.

At least for now, merge the null cases into the empty string cases so we can survive import.

Test Plan
  • Constructed a commit with no author and no commit message using the approach described in T13538; pushed and parsed it.
  • Before: fatals during identity selection and storing the commit message (both roughly NULL inserts into non-null columns).
  • After: clean import.

This produces a less-than-ideal UI in Diffusion, but it doesn't break anything:

Screen Shot 2020-05-18 at 7.59.36 PM.png (307×516 px, 22 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable