Add index for classification attribute#35459
Add index for classification attribute#35459akhil1508 wants to merge 1 commit intonextcloud:masterfrom
Conversation
0621877 to
e6c4d89
Compare
4f56a96 to
25ff090
Compare
tcitworld
left a comment
There was a problem hiding this comment.
You also need to bump the app version in apps/dav/appinfo/info.xml so that the migrations are executed.
|
How long did creating the index took on your database? If it's too much we can move it to the dedicated missing indexes command (see for instance #33405) |
|
@tcitworld It took 14 seconds to create the index on a database with 1 million entries in the |
839c1d5 to
1d38939
Compare
|
@tcitworld Bumped from 1.25.0 to 1.25.1 |
a856cbc to
b08e885
Compare
0cd8125 to
57ec1fb
Compare
|
@akhil1508 it should work with: then mark all of you commits as "s" (for squash) and it should work. |
beccaea to
604f141
Compare
Signed-off-by: Akhil <akhil@e.email>
604f141 to
33fd502
Compare
|
@susnux I did as you suggested and force-pushed, now there are 33 changes instead of 3 changes 🤔 |
|
@akhil1508 Is the « Allow edits by maintainers » checkbox checked on the sidebar? Seems I can't force push to fix things. |
|
@tcitworld Seems I cannot do that as this PR is made from a fork under an organization :( As per https://github.com/orgs/community/discussions/5634 |
|
Replacement PR at #41111 |
|
@tcitworld Thanks for the replacement PR :) |
Summary
TODO
Checklist