Bug 767205 - Agent performs excessive IO when sending drift content to server
Summary: Agent performs excessive IO when sending drift content to server
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: RHQ Project
Classification: Other
Component: Agent, drift
Version: 4.2
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: RHQ 4.3.0
Assignee: John Sanda
QA Contact: Mike Foley
URL:
Whiteboard:
Depends On: jon30-sprint10, rhq43-sprint10
Blocks: 707225 785847
TreeView+ depends on / blocked
 
Reported: 2011-12-13 14:40 UTC by John Sanda
Modified: 2013-08-31 10:15 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
: 785847 (view as bug list)
Environment:
Last Closed: 2013-08-31 10:15:33 UTC
Embargoed:


Attachments (Terms of Use)

Description John Sanda 2011-12-13 14:40:44 UTC
Description of problem:
When the server requests content for files previously reported, the agent receives a list of SHAs. The agent then performs a reverse look up, finding the corresponding file system path for each SHA. This involves reading the current snapshot file. The problem is that the snapshot file is read over and over again for each SHA. When the server sends a lot of SHAs, like with the initial snapshot, this results in excessive and unnecessary IO. The agent only needs to read the snapshot file once. Doing so will increase the memory usage for the duration of that method call; however, the overall execution time for the method will substantially be reduced as the number of SHAs passed to it increases in size. The increased memory footprint is likely on the order of bytes or 10s of kilobytes whereas the decrease in execution time could be on the order of milliseconds or seconds.

For testing, this should result in no change in functionality other than increased logging.

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:

Comment 1 John Sanda 2011-12-15 02:01:47 UTC
DriftFilesSender has been refactored so that it only reads the snapshot file on disk once when looking up files by SHA.

master commit hash: 9f18aee38ed44f992ce26e7f8141a057af716533

Comment 2 Mike Foley 2011-12-15 19:11:48 UTC
verified thru ad hoc testing on drift feature

Comment 3 Charles Crouch 2012-01-30 18:56:04 UTC
Setting Target release correctly to RHQ4.3, since it was tested in master

Comment 4 Heiko W. Rupp 2013-08-31 10:15:33 UTC
Bulk close of old bugs in VERIFIED state.


Note You need to log in before you can comment on or make changes to this bug.