New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Turn off strict mode on import #3784
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
Just one documentation suggestion.
Co-authored-by: Clayton Liddell <clayton.liddell@civicactions.com>
Hmm committed your suggestions @clayliddell but seeing it's failing one of the PHPunit runs, but not the other. Weird, will take another look. |
…nodb-create-table
This reverts commit 8750aa8.
a8fdffb
to
005801e
Compare
Moved this into the datastore's DatabaseTable class, after realizing that's where the table creation is actually happening. |
1 similar comment
Moved this into the datastore's DatabaseTable class, after realizing that's where the table creation is actually happening. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just 1 small thing. Trying not to be picky, but I wanted to call this out.
Strict mode on innodb makes it hard to create tables over 200 columns, which has come up in real world use. This sets strict mode off simply at the session level for the datastore, and keeps it on for the rest of Drupal's operation.
This has been tested locally and successfully created a 252 column table in the same environment where it had previously failed.