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

Bug 329334

Summary: [Backport] Concurrent access to file while decorating
Product: [Eclipse Project] Platform Reporter: Szymon Ptaszkiewicz <sptaszkiewicz>
Component: UIAssignee: Platform UI Triaged <platform-ui-triaged>
Status: RESOLVED FIXED QA Contact: Oleg Besedin <ob1.eclipse>
Severity: normal    
Priority: P3 CC: bokowski, ob1.eclipse, pwebster, remy.suen, Szymon.Brandys, tomasz.zarna
Version: 3.5.2   
Target Milestone: 3.5.2+   
Hardware: All   
OS: All   
Whiteboard:
Bug Depends on: 327192    
Bug Blocks:    
Attachments:
Description Flags
Patch v.0.1 none

Description Szymon Ptaszkiewicz CLA 2010-11-03 07:56:13 EDT 
Bug cloned to track potential release into 3.5.x maintenance branch.

+++ This bug was initially created as a clone of Bug #327192 +++

The issue came out while working on bug 301223. The problem is that during project set import decorating thread and importing thread are both trying to use the same file. Decorating thread is calling file.getContentDescription() and importing thread is trying to delete this file to make space for incoming file from repository. The file is open because of decorator so delete operation is not successful which results in error during import. Delete operation uses scheduling rule but decorator is not. There should be added simple scheduling rule to decorate operation to prevent concurrent access and error during import.
Comment 1 Szymon Ptaszkiewicz CLA 2010-11-03 12:04:34 EDT 
Created attachment 182306 [details]
Patch v.0.1
Comment 2 Boris Bokowski CLA 2010-11-15 17:05:16 EST 
+1 for 3.5.2
Comment 3 Boris Bokowski CLA 2010-11-15 17:05:40 EST 
I meant +1 for 3.5.2+ :-)
Comment 4 Oleg Besedin CLA 2010-11-16 15:54:54 EST 
Patch applied to the "R3_5_maintenance" branch. I also incremented bundle version from 3.5.2 to 3.5.3. The bundle is tagged and UI map file is updated.