Skip to content

[stable33] perf(preview): Optimize retriving all previews from oc_filecache - #62603

Open
backportbot[bot] wants to merge 6 commits into
stable33from
backport/62601/stable33
Open

[stable33] perf(preview): Optimize retriving all previews from oc_filecache#62603
backportbot[bot] wants to merge 6 commits into
stable33from
backport/62601/stable33

Conversation

@backportbot

@backportbot backportbot Bot commented Jul 28, 2026

Copy link
Copy Markdown

Backport of #62601

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Review and verify the backported changes
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@backportbot
backportbot Bot requested a review from CarlSchwan July 28, 2026 08:45
@backportbot backportbot Bot added this to the Nextcloud 33.0.8 milestone Jul 28, 2026
@CarlSchwan
CarlSchwan force-pushed the backport/62601/stable33 branch from 8934f1c to 1261cb4 Compare July 28, 2026 08:49
@CarlSchwan
CarlSchwan marked this pull request as ready for review July 28, 2026 08:51
@CarlSchwan
CarlSchwan requested a review from a team as a code owner July 28, 2026 08:51
@CarlSchwan
CarlSchwan requested review from come-nc, leftybournes and provokateurin and removed request for a team July 28, 2026 08:51
Previoulsy we used a PATH LIKE expression which is fine for small
instances but doesn't scale for big instance (timeout).

Manually moving accross the tree with getFolderContentsById is
significantly faster as we can use the index and this also reuse common
APIs from OCP/Files instead of directly manipulating the filecache with
the query builder.

Assisted-by: ClaudeCode:claude-opus-4-8
Signed-off-by: Carl Schwan <carl@carlschwan.eu>
@CarlSchwan
CarlSchwan force-pushed the backport/62601/stable33 branch from 1261cb4 to 2b7c149 Compare July 28, 2026 10:14
Faster than doing a commit after each file

Assisted-by: ClaudeCode:claude-opus-4-8
Signed-off-by: Carl Schwan <carl@carlschwan.eu>
Assisted-by: ClaudeCode:claude-opus-4-8
Signed-off-by: Carl Schwan <carl@carlschwan.eu>
Assisted-by: ClaudeCode:claude-opus-4-8
Signed-off-by: Carl Schwan <carl@carlschwan.eu>
Signed-off-by: Carl Schwan <carl@carlschwan.eu>
Signed-off-by: Carl Schwan <carl@carlschwan.eu>
@CarlSchwan
CarlSchwan requested a review from artonge July 28, 2026 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants