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

Bug 366066

Summary: Double refresh breaks model consistency
Product: [Modeling] EMF Reporter: Eike Stepper <stepper>
Component: cdo.coreAssignee: Egidijus Vaisnora <vaisegid>
Status: CLOSED FIXED QA Contact: Eike Stepper <stepper>
Severity: normal    
Priority: P3    
Version: 4.0   
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:
Attachments:
Description Flags
attached patch none

Description Eike Stepper CLA 2011-12-08 12:13:31 EST 
Cloned from: 361819: Double refresh breaks model consistency
https://bugs.eclipse.org/bugs/show_bug.cgi?id=361819

It happens in cases, when server side contains objects with greater version by value of two or more comparing to local version. When two refresh are called on the session, first refresh brings newer revision adds to local cache and converts object to PROXY state, second updates removes newer revision from local cache.  PROXY state is resolved with a revision, found in the local cache (it is the first version revision).
Comment 1 Eike Stepper CLA 2011-12-11 04:02:11 EST 
Hi Egidijus, I thought I did merge (cherry pick) bug 361819 fix from 4.1 but something must have gone wrong. The tests are now failing in 4.0. Probably because some changes in CDORevisionManagerImpl are missing. Can you please investgiate?
Comment 2 Egidijus Vaisnora CLA 2011-12-12 08:43:32 EST 
Created attachment 208258 [details]
attached patch

RevisionManager was not included into commit. Attached patch contains missing part, but I did not commit it. For some reasons I cannot run whole test - tests from bugzilla package are not collected into test suite.
((AbstractBundle)bundle).getClasses() gives empty list
Comment 3 Eike Stepper CLA 2011-12-15 13:55:38 EST 
Your patch did not contain the right changes but I got them from master.

commit 046111e9823a749eff20516bcbeb404a4b5f2ce2
Comment 4 Eike Stepper CLA 2012-09-21 06:51:13 EDT 
Closing.