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

chore(filecache): remove index fs_id_storage_size #50791

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Altahrim
Copy link
Collaborator

@Altahrim Altahrim commented Feb 13, 2025

Summary

Removes index fs_id_storage_size
This index was introduced in #29118 to optimize incomplete scan search.
The request is modified in #50781 and the index isn't necessary anymore.

TODO

Checklist

@Altahrim Altahrim added the 3. to review Waiting for reviews label Feb 13, 2025
@Altahrim Altahrim added this to the Nextcloud 32 milestone Feb 13, 2025
@Altahrim Altahrim requested review from nickvergessen and a team February 13, 2025 09:57
@Altahrim Altahrim self-assigned this Feb 13, 2025
@Altahrim Altahrim requested review from artonge, nfebe and sorbaugh and removed request for a team February 13, 2025 09:57
@Altahrim Altahrim marked this pull request as draft February 13, 2025 09:59
@Altahrim Altahrim added 2. developing Work in progress and removed 3. to review Waiting for reviews labels Feb 13, 2025
@Altahrim Altahrim force-pushed the perf/remove-filecache-index branch 2 times, most recently from ba26160 to b6b7e2f Compare February 13, 2025 10:27
Index is outdated by PR 50781

Signed-off-by: Benjamin Gaussorgues <[email protected]>
@Altahrim Altahrim force-pushed the perf/remove-filecache-index branch from b6b7e2f to 3548bde Compare February 13, 2025 10:37
@Altahrim Altahrim added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Feb 13, 2025
@Altahrim Altahrim marked this pull request as ready for review February 13, 2025 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants