-
Couldn't load subscription status.
- Fork 287
Performance improvements migration #6234
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
Conversation
29f1721 to
873e34a
Compare
|
/backport to stable1.12 |
873e34a to
a2a4c05
Compare
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.
Tested locally and it works
a2a4c05 to
e2db9cf
Compare
|
info.xml bump missing |
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.
|
@miaulalala could you bump info.xml, rename the migration and rebase to latest main? |
change columnst to bigint, create named foreign keys Signed-off-by: Anna Larch <anna@nextcloud.com>
e2db9cf to
f381db7
Compare
|
@miaulalala can we close #5294 too? |
|
Yep, that should be fine to close. |

Add indices on mail_messages table
change column types to bigint for tables recipients and messages
Change foreign keys to named foreign keys for recipients and attachments
Fixes #4466
Fixes #5985
Fixes #4664
Fixes #6223