Closed Bug 913817 Opened 11 years ago Closed 11 years ago

HTTP cache v2: check how the file deleter loop break works

Categories

(Core :: Networking: Cache, defect)

Product:
Core
Component:
Networking: Cache
Type:
defect
Priority:
Not set
Severity:
minor

Tracking

()

Status:
RESOLVED INVALID

People

(Reporter: mayhemer, Assigned: mayhemer)

References

(Blocks 1 open bug)

Details

(Whiteboard: [cache2])

The file deleter is a class that iterates all files in /entries dir and deletes them one by one. For optimization reasons not to block the I/O thread but still be efficient I break the loop after some short time. However, after recent inspection of the code and also because there is no test for this it might be that this break logic is not perfect. Also, a completely different approach to file deleting can be taken.
[ No longer blocks enabling cache2, expected to be fixed after cache2 is on ]
Blocks: cache2afterenable
No longer blocks: cache2enable
Blocks: cache2enable
No longer blocks: cache2enable
Blocks: cache2enable
No longer blocks: cache2enable
No longer applies. The code has been removed. Anyway, today we have YieldAndRerun.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.