Skip to content
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

Use FileInfo's metadata for hidden prop #42349

Merged
merged 1 commit into from
Dec 19, 2023
Merged

Conversation

artonge
Copy link
Contributor

@artonge artonge commented Dec 18, 2023

This prevents a SQL request on files_metadata on each requested files.

Part of #42346

@artonge artonge force-pushed the artonge/perf/dav_hidden_prop branch from daa1d05 to 905be58 Compare December 18, 2023 10:47
@artonge artonge requested review from ArtificialOwl, a team, icewind1991, nfebe and come-nc and removed request for a team, icewind1991 and nfebe December 18, 2023 10:48
@artonge artonge force-pushed the artonge/perf/dav_hidden_prop branch from 905be58 to 0f1a84c Compare December 18, 2023 11:54
@artonge
Copy link
Contributor Author

artonge commented Dec 18, 2023

/backport to stable28

@artonge
Copy link
Contributor Author

artonge commented Dec 18, 2023

CI failure unrelated

@artonge artonge added this to the Nextcloud 29 milestone Dec 18, 2023
@artonge artonge self-assigned this Dec 18, 2023
@artonge artonge added 3. to review Waiting for reviews performance 🚀 labels Dec 18, 2023
@artonge artonge force-pushed the artonge/perf/dav_hidden_prop branch from 0f1a84c to 253c3f2 Compare December 18, 2023 14:54
@artonge artonge force-pushed the artonge/perf/dav_hidden_prop branch from 253c3f2 to 6948c07 Compare December 19, 2023 09:23
@artonge artonge requested review from a team, icewind1991, sorbaugh and Altahrim and removed request for a team and icewind1991 December 19, 2023 09:23
@artonge artonge enabled auto-merge December 19, 2023 10:33
@artonge artonge merged commit 2f647aa into master Dec 19, 2023
50 checks passed
@artonge artonge deleted the artonge/perf/dav_hidden_prop branch December 19, 2023 13:17
@blizzz blizzz mentioned this pull request Mar 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants