Bug 72304 - [Core Performance] Add to .cvsignore does not scale well
Summary: [Core Performance] Add to .cvsignore does not scale well
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: CVS (show other bugs)
Version: 3.0   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.1 M7   Edit
Assignee: platform-cvs-inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords: performance
Depends on:
Blocks:
 
Reported: 2004-08-19 16:19 EDT by Oyvind Harboe CLA
Modified: 2005-05-10 10:20 EDT (History)
1 user (show)

See Also:


Attachments
shows unresponsive GUI during add to .cvsignore (154.05 KB, image/pjpeg)
2004-08-19 16:23 EDT, Oyvind Harboe CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Oyvind Harboe CLA 2004-08-19 16:19:45 EDT
I added >1000 files to .cvsignore and the userinterface was unresponsive for 
~60 seconds.

The application was at 100% usage and the rest of windows was also stuck. 
Seems like adding to .cvsignore held up the Windows event loop or something.
Comment 1 Oyvind Harboe CLA 2004-08-19 16:23:51 EDT
Created attachment 14082 [details]
shows unresponsive GUI during add to .cvsignore
Comment 2 Michael Valenta CLA 2004-08-25 14:51:23 EDT
I see two problems in this report

1) CVS is adding the ignores in a busy-cursor whereas it should be done in a 
progress monitor or, better yet, a user job.

2) In the screenshow provided, the add to ignore list is adding an entire 
hierarchy when adding the parents would be enough.

Having said that, point 2 doesn't cause a problem since the .cvsignore file is 
properly added to each directory.

Point 1 should be addressed by making the ignore action a user job that is run 
after OK is pressed on the ignore dialog.
Comment 3 Michael Valenta CLA 2004-08-25 15:41:29 EDT
I added the use of a background job but the problem still remains (but is less 
severe). The problem is that the CVS meta-data synchronizer does not batch 
changes for ignore files. Also, I noticed that removing an ignore from a 
parent will cause it's direct children to appear in the synchronize view but 
indirect descendants will not be readded.
Comment 4 Tod Creasey CLA 2005-03-07 11:57:25 EST
Adding my name to the cc list as we are now tracking performance issues more
closely. Please remove the performance keyword if this is not a performance bug.
Comment 5 Michael Valenta CLA 2005-04-06 16:33:06 EDT
Fix released to HEAD. UI is no longer blocked and reverting an ignore works 
deeply now.
Comment 6 Tod Creasey CLA 2005-04-11 10:05:24 EDT
Michael can you tell me the performance improvement this represents for an
average page (so that we can show it on the performance page)?
Comment 7 Michael Valenta CLA 2005-04-11 10:14:45 EDT
The case stated in the original comment should now only take a few seconds and 
since it is in the background, the UI is not blocked at all.
Comment 8 Michael Valenta CLA 2005-05-10 10:20:26 EDT
Verified