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.

Sign up for GitHub

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

Jump to bottom

Use CappedCache for normalized file names #24340

Closed
PVince81 opened this issue Apr 29, 2016 · 3 comments · Fixed by #24351
Closed

Use CappedCache for normalized file names #24340

PVince81 opened this issue Apr 29, 2016 · 3 comments · Fixed by #24351
Assignees
@icewind1991
Labels
performance Type:Bug
Milestone
9.1

Comments

@PVince81
Copy link
Contributor

Could help resolve memory errors when scanning a bit number of file.

@icewind1991
@PVince81 PVince81 added this to the 9.1-current milestone Apr 29, 2016
@PVince81
Copy link
Contributor Author

@icewind1991 mind having a look ?

@PVince81
Copy link
Contributor Author

On a side note I saw with a debugger what kind of crazy names are going there in the "cache missed" portion of the method. Sometimes there is a leading slash, sometimes not, sometimes two, for the same name coming up several times. If that happens for every entry, it is likely that it will use even more memory.

@icewind1991 icewind1991 mentioned this issue Apr 29, 2016
Merged
@lock
Copy link

lock bot commented Aug 4, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Aug 4, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@icewind1991 icewind1991

Labels
performance Type:Bug
Projects
None yet
Milestone
9.1
Development

Successfully merging a pull request may close this issue.

cap the normalized path cache owncloud/core
2 participants
@PVince81 @icewind1991