Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 355977

Summary: Adding review items from SVN is really slow
Product: z_Archived Reporter: Sebastien Dubois <sebastien.dubois>
Component: MylynAssignee: Project Inbox <r4e-triaged>
Status: RESOLVED FIXED QA Contact:
Severity: enhancement    
Priority: P3 CC: alvaro.sanchez-leon, lmcbout
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows Vista   
Whiteboard:

Description Sebastien Dubois CLA 2011-08-26 15:40:13 EDT 
When adding new review items using the new SVN Subclipse connector, it takes a very long time to add the review item from the remote SVN repository.  This needs to be investigated.  

At first glance, it seems that the problem is related to the fetching of the files.

As a side note, it would be good to put more details in the Eclipse Job Progress window to give more information on the sub-actions that are done when adding the review items (e.g. fetching file, copying file, calculating differences, adding data to model etc.)
Comment 1 Sebastien Dubois CLA 2011-09-07 15:24:17 EDT 
We identified that the fetching of the files from the SVN repository takes almost all the time needed to execute the command.  Computing differences and populating the model takes less than 2% of the total time.

So a good way to improve performance would be to fetch all files from the remote repository in separate threads in parallel.  However, this introduces more complexity in managing the thread and making sure that they don't collide when gathering data and writing information in the model.  In any case, this implementation is not trivial and should be investigated as an enhancement later on.
Comment 2 Sebastien Dubois CLA 2011-09-07 15:25:05 EDT 
As a side note, for now I implemented subtasks for the Find Review Items command so that the user has a better idea of what is going on.
Comment 3 Sebastien Dubois CLA 2013-01-24 16:34:20 EST 
THis functionality was implemented as of now.  This can be fixed