In Drupal Gardens we saw someone use form builder to make a webform that was big enough to cause the {form_builder_cache} table to overflow, leading to fatal errors.

The form itself was large, but not gigantic - maybe 30 fields or so.

Drupal core uses 'longblob' columns (rather than 'blob') for its cache tables, which pretty much guarantees they will never fill up. The attached patch switches Form Builder to use 'longblob' columns also.

Comments

quicksketch’s picture

Sounds good. I'd like an update function, but considering the small usage of Form Builder we can probably let that slide.

David_Rothstein’s picture

StatusFileSize
new920 bytes

Ah! I figured you wouldn't want to commit an update function, since the module is still in development and not officially supported yet.

But I already wrote one as custom code anyway, so it's certainly easy to add that to the patch. Here it is with that.

David_Rothstein’s picture

The new patch only applies to Drupal 7, of course (whereas the other should apply to D6 also).

quicksketch’s picture

StatusFileSize
new723 bytes
new764 bytes

Committed this pair of patches.

quicksketch’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.