Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 366066 - Double refresh breaks model consistency
Summary: Double refresh breaks model consistency
Status: CLOSED FIXED
Alias: None
Product: EMF
Classification: Modeling
Component: cdo.core (show other bugs)
Version: 4.0   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Egidijus Vaisnora CLA
QA Contact: Eike Stepper CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-12-08 12:13 EST by Eike Stepper CLA
Modified: 2012-09-21 06:51 EDT (History)
0 users

See Also:

Attachments
attached patch (3.80 KB, text/plain)
2011-12-12 08:43 EST, Egidijus Vaisnora CLA 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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.