-
-
Notifications
You must be signed in to change notification settings - Fork 212
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
Only request the needed schema in table view #2568
Only request the needed schema in table view #2568
Conversation
This changes uses the array returned by $section->fetchVisibleColumns() in order to create the element names array to pass to the entry manager. This yeilds a nice performance update when a section contains many (hundreads) of fields but only a few of them are needed for the publish table view.
Honestly, I don't see any performance difference. Should I increase the number of entries per page, maybe? |
Yes. But you get a real gain when you have a field with hundreds of fields (which, I agree, can be a pain to create). But most importantly: there's no bug? |
No, it didn't break, which is good. :-)
My test section has 75 fields — no visible difference, even with 100 entries per page. Maybe my server is too fast. :-)) |
Well, maybe :P The problem was not the webserver tho, it was the DB... I saved 500ms of a request that was taking 2500ms (which is 20% decrease) |
After testing by myself and @michael-e it's safe to merge in. |
This changes uses the array returned by $section->fetchVisibleColumns() in order to create the element names array to pass to the entry manager. This yeilds a nice performance update when a section contains many (hundreads) of fields but only a few of them are needed for the publish table view. Rebase of 34bd49f
This changes uses the array returned by $section->fetchVisibleColumns() in order to create the element names array to pass to the entry manager. This yeilds a nice performance update when a section contains many (hundreads) of fields but only a few of them are needed for the publish table view. Rebase of 34bd49f Picked from 92f762b
This changes uses the array returned by $section->fetchVisibleColumns() in order to create the element names array to pass to the entry manager. This yeilds a nice performance update when a section contains many (hundreads) of fields but only a few of them are needed for the publish table view. Rebase of 34bd49f Picked from 92f762b
This changes uses the array returned by $section->fetchVisibleColumns() in order to create the element names array to pass to the entry manager. This yields a nice performance update when a section contains many (hundreds) of fields but only a few of them are needed for the publish table view.
I do not see how this can't be backward compatible, but I created a PR just to document the change and ask for testers! :)
Thanks!