perf(sync): Reduce db operations for accounts without CONDSTORE

[JohannesGGE]

Fix: #6341

---

Queries (only related) with open mail app for 40min (approx. 170 mails in the mailbox)

before:

1367 	 primary UPDATE `oc_mail_messages` SET `flag_answered` = :flag_answered, `flag_deleted` = :flag_deleted, `flag_draft` = :flag_draft, `flag_flagged` = :flag_flagged, `flag_seen` = :flag_seen, `flag_forwarded` = :flag_forwarded, `flag_junk` = :flag_junk, `flag_notjunk` = :flag_notjunk, `flag_mdnsent` = :flag_mdnsent, `flag_important` = :flag_important, `updated_at` = :dcValue1 WHERE (`uid` = :uid) AND (`mailbox_id` = :mailbox_id)

after:
(So usually there are 2 queries per flag (138) (set true and set false). seen and notjunk are the only ones with doubled queries because for the other flags, either all mails or no mail have the flag in this specific mailbox.)

    138 	 primary UPDATE `oc_mail_messages` SET `flag_seen` = :dcValue1, `updated_at` = :dcValue2 WHERE (`uid` IN (:uids)) AND (`mailbox_id` = :dcValue3)
    138 	 primary UPDATE `oc_mail_messages` SET `flag_notjunk` = :dcValue1, `updated_at` = :dcValue2 WHERE (`uid` IN (:uids)) AND (`mailbox_id` = :dcValue3)
     69 	 primary UPDATE `oc_mail_messages` SET `flag_mdnsent` = :dcValue1, `updated_at` = :dcValue2 WHERE (`uid` IN (:uids)) AND (`mailbox_id` = :dcValue3)
     69 	 primary UPDATE `oc_mail_messages` SET `flag_junk` = :dcValue1, `updated_at` = :dcValue2 WHERE (`uid` IN (:uids)) AND (`mailbox_id` = :dcValue3)
     69 	 primary UPDATE `oc_mail_messages` SET `flag_important` = :dcValue1, `updated_at` = :dcValue2 WHERE (`uid` IN (:uids)) AND (`mailbox_id` = :dcValue3)
     69 	 primary UPDATE `oc_mail_messages` SET `flag_forwarded` = :dcValue1, `updated_at` = :dcValue2 WHERE (`uid` IN (:uids)) AND (`mailbox_id` = :dcValue3)
     69 	 primary UPDATE `oc_mail_messages` SET `flag_flagged` = :dcValue1, `updated_at` = :dcValue2 WHERE (`uid` IN (:uids)) AND (`mailbox_id` = :dcValue3)
     69 	 primary UPDATE `oc_mail_messages` SET `flag_draft` = :dcValue1, `updated_at` = :dcValue2 WHERE (`uid` IN (:uids)) AND (`mailbox_id` = :dcValue3)
     69 	 primary UPDATE `oc_mail_messages` SET `flag_deleted` = :dcValue1, `updated_at` = :dcValue2 WHERE (`uid` IN (:uids)) AND (`mailbox_id` = :dcValue3)
     69 	 primary UPDATE `oc_mail_messages` SET `flag_answered` = :dcValue1, `updated_at` = :dcValue2 WHERE (`uid` IN (:uids)) AND (`mailbox_id` = :dcValue3)

[ChristophWurst]

The speedup is unparalleled

https://blackfire.io/profiles/compare/e2ee40f6-f178-401e-a403-f1f791907144/graph

🚀

[ChristophWurst]

to experiment with: AND flag=1

to lower the number of accessed rows and hopefully the transaction size (for rollbacks and for replication)

[ChristophWurst]

Seems worth pursuing

Diff to the current HEAD

https://blackfire.io/profiles/compare/797f6164-4438-4c43-bcde-8d18dbd18b76/graph

Diff to main

https://blackfire.io/profiles/compare/eaa6c793-1f5b-4793-b13d-ea14a5b9ca20/graph

[JohannesGGE]

✅

[ChristophWurst]

Please use modern array syntax

🚀

[github-actions]

Hello there,
Thank you so much for taking the time and effort to create a pull request to our Nextcloud project.

We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process.

Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6

Thank you for contributing to Nextcloud and we hope to hear from you soon!