[codeface] Re: [PATCH] Follow-up patch for my latest patch stack

  • From: Wolfgang Mauerer <wolfgang.mauerer@xxxxxxxxxxxxxxxxx>
  • To: <codeface@xxxxxxxxxxxxx>
  • Date: Tue, 6 Dec 2016 15:22:07 +0100

Hi Mitchell,

Am 06/12/2016 um 14:48 schrieb Claus Hunsen:

While analyzing Linux on our cluster, we observed that the commit analysis
breaks due to an oversized commit message:
2016-12-02 08:43:18 [codeface.dbmanager] Process-8 CRITICAL: MySQL error
1406 during "INSERT INTO commit (AddedLines, AuthorSubsysSimilarity,
AuthorTaggersSimilarity, ChangedFiles, CmtMsgBytes, CmtMsgLines,
DeletedLines, DiffSize, NumSignedOffs, NumTags, Subsys,
TaggersSubsysSimilarity, TotalSubsys, author, authorDate,
authorTimeOffset, commitDate, commitHash, corrective, description, inRC,
projectId, releaseRangeId) VALUES (%s, %s, %s, %s, %s, %s, %s, %s, %s,
%s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s, %s)": Data too long
for column 'description' at row 8022

To prevent this error, I increased the data type to "MEDIUMTEXT" (in
comparison to "TEXT" before).

I would like if the appended patch is incorporated with the remainder of
my previously posted patch stack while merging. The patch is already
pushed to my 'claus-updates' branch on GitHub.

I seem to recall that Mitchell did some similar patch a while ago -- can
you please try if this change is already included in branch "mergetest"
or "conway-review"? I'd like to make sure we're not fixing the same
issue twice.

Thanks, Wolfgang

Other related posts: