layout_builder__layout_section column hitting database limit

+  public function testUpdateHookN() {
+    $this->runUpdates();
+    $this->assertEqualLayoutSectionDataType('longblob');
+  }

Can we add an assertion before the updates run. To test for what it is now.

Addressed the Comment #55
Attached patch against Drupal 10.1.x

#56 isn't quite doing what #55 asked for, but it also shows that the test isn't testing what it thinks it is, since it'll pass even without the update running.

Addressed #55 adding an assertion after to the check the expected value after test run.

Per #58 the tests should be looked at as they don't appear to be testing what's expected.

hii, The above patch failed because the branch has been updated with file layout-builder.php, so removing the file from the patch and adding assertion after to the check the expected value after test run, addressed #55.

indiff is same as #60

Attaching interdiff, thanks.

The last submitted patch, 62: 3030154-62.patch, failed testing. View results →

Patch #42 works very well for us on D9.5.

We are using a heavily customized layout builder setup though.
It contains inline block_content for section configuration through https://www.drupal.org/project/layout_block →
It also contains a few dozen block_content types for actual section content.
All of these blocks have 50+ fields on them. (some of them coming close to 100 fields)
We only had issues when using the section_library module. Related issue that reference me here: https://www.drupal.org/project/section_library/issues/3232071 →

Hi, could this patch considered as "safe" for production environment or is still under development? Thank you very much.

@FiNeX FWIW, we've been using this in production for a while now and I would consider it safe.

✨ Additional blob field sizes in schema definitions Needs work is now in RTBC. If it lands before this, I could easily see us switching from a blob:big to blob:medium.

This is ready for review again after a rebase + some changes. Tests pass green.

We should postpone this one then.

But pipeline is green so that's good and probably an easy update when the other lands.