Task #59771
closedChange be_sessions:ses_data field from LONGTEXT to TEXT
0%
Description
The ses_data field of the be_sessions table has always been LONGTEXT.
This breaks TYPO3 sessions when running under hhvm: https://notes.typo3.org/p/hhvm
The field started out as a blob field. Later it changed to a mediumblob, then into a mediumtext before finally turning into a longtext back in 2007.
So great! Now the field can hold 4GB of data :-)
But it breaks hhvm.
We can safely change it to a text field. That will hold 64K of data. And as you all know even 4k should be enough for everyone: http://blog.hvidtfeldts.net/index.php/2007/11/4k-should-be-enough-for-everyone/
But all jokes aside, the field may hold 21.844 in a worst case (3 byte UTF-8 characters: http://stackoverflow.com/questions/4420164/how-much-utf-8-text-fits-in-a-mysql-text-field).
Updated by Jose Antonio Guerra over 10 years ago
See related info on GitHub issue (read this comment and below): https://github.com/facebook/hhvm/issues/2377#issuecomment-46751747
Updated by Gerrit Code Review almost 10 years ago
- Status changed from New to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/35732
Updated by Wouter Wolters almost 10 years ago
- Status changed from Under Review to Resolved
Updated by Markus Klein almost 10 years ago
- Status changed from Resolved to Rejected
The patch was reverted with https://review.typo3.org/35762
The ticket is rejected as the storage size is needed for workspaces, which uses quite some session memory.