Task #858 (closed)
Opened 16 years ago
Closed 16 years ago
RepositoryInfo.getFreeSpaceInKb() et al. asynchronous checks to save time on file writes
Reported by: | jamoore | Owned by: | jamoore |
---|---|---|---|
Priority: | major | Milestone: | 3.0-Beta3.1 |
Component: | Performance | Version: | 3.0-M1 |
Keywords: | REVIEW | Cc: | cxallan |
Resources: | n.a. | Referenced By: | n.a. |
References: | n.a. | Remaining Time: | n.a. |
Sprint: | n.a. |
Description
Currently on every OriginalFile or Pixels write, a check is being made of disk space. The original plan for the service would be that it would keep up with the available space via polling or notifications, and the getFreeSpace.. methods would simply return a cached value.
Based on the performance penalty (currently measured), it might be important to get this out of the time critical write method paths.
Change History (1)
comment:1 Changed 16 years ago by jmoore
- Keywords REVIEW added
- Milestone changed from 3.0-Beta4 to 3.0-Beta3.1
- Resolution set to fixed
- Status changed from new to closed
Done as part of ScalingOmero in r2675. This fix penalizes one thread periodically with the task of calculating the remaining time. As mentioned in the [r2675 commit log] if needed, an asychronous thread can be added to penalize fewer if any user threads. Closing for now.