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

[stable10] Disable background scan for home storage/cache #29306

Merged

Conversation

DeepDiver1975
Copy link
Member

The home cache might have entries with size=-1 that are outside of
"files/" due to encryption or other metadata that generally do not
require the size to be set or accurate.

Instead of adjusting the HomeCache to ignore entries outside of
"files/", let's just disable it completely. The home storage is in the
data folder and must not receive external changes, so it should never
have any entries with size=-1 in "files/".

backport of #28987 to stable10

The home cache might have entries with size=-1 that are outside of
"files/" due to encryption or other metadata that generally do not
require the size to be set or accurate.

Instead of adjusting the HomeCache to ignore entries outside of
"files/", let's just disable it completely. The home storage is in the
data folder and must not receive external changes, so it should never
have any entries with size=-1 in "files/".
@DeepDiver1975 DeepDiver1975 added this to the development milestone Oct 19, 2017
@DeepDiver1975 DeepDiver1975 self-assigned this Oct 19, 2017
@codecov
Copy link

codecov bot commented Oct 20, 2017

Codecov Report

Merging #29306 into stable10 will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@              Coverage Diff               @@
##             stable10   #29306      +/-   ##
==============================================
+ Coverage        58.4%    58.4%   +<.01%     
- Complexity      17693    17694       +1     
==============================================
  Files            1059     1059              
  Lines           59341    59343       +2     
==============================================
+ Hits            34657    34659       +2     
  Misses          24684    24684
Impacted Files Coverage Δ Complexity Δ
lib/private/Files/Cache/HomeCache.php 100% <100%> (ø) 18 <1> (+1) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ef290c2...d76dca3. Read the comment docs.

Copy link
Contributor

@PVince81 PVince81 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@PVince81 PVince81 merged commit a2eb09d into stable10 Oct 20, 2017
@PVince81 PVince81 deleted the stable10-343af394eca914b610aa61ff5e201e2e93a58ca7 branch October 20, 2017 15:28
@lock
Copy link

lock bot commented Aug 2, 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 2, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants