Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 343611 - Merging ObjectChangeSets fail to merge deferredSet
Summary: Merging ObjectChangeSets fail to merge deferredSet
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Eclipselink (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Gordon Yorke CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-04-21 15:29 EDT by Gordon Yorke CLA
Modified: 2022-06-09 10:04 EDT (History)
1 user (show)

See Also:


Attachments
Proposed Patch (5.76 KB, patch)
2011-04-21 16:02 EDT, Gordon Yorke CLA
no flags Details | Diff
Tests (1.08 KB, patch)
2011-04-21 16:21 EDT, Gordon Yorke CLA
no flags Details | Diff
Updated Patch (11.18 KB, patch)
2011-04-27 12:08 EDT, Gordon Yorke CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Gordon Yorke CLA 2011-04-21 15:29:59 EDT
In cased where an ObjectChangeSet needs to be merged (after commitAndResume for instance) any tracking of the set of deferred collections is not merged and the information will be lost.
Comment 1 Gordon Yorke CLA 2011-04-21 15:41:00 EDT
This will result in any changes made after a UnitOfWork resume being lost when an entire collection was replaced.
Comment 2 Gordon Yorke CLA 2011-04-21 15:43:09 EDT
UnitOfWorkResumeOnFailureTest has been updated to actually verify the changes and not just compare the merged object with the database object.
Comment 3 Gordon Yorke CLA 2011-04-21 16:02:47 EDT
Created attachment 193876 [details]
Proposed Patch

This patch causes the deferred set to be merged.
Comment 4 Gordon Yorke CLA 2011-04-21 16:21:54 EDT
Created attachment 193884 [details]
Tests
Comment 5 Gordon Yorke CLA 2011-04-27 12:08:43 EDT
Created attachment 194177 [details]
Updated Patch

This patch includes some updates to support the ObjectChangeSet merge that occurs after the flush of a PersistenceContext
Comment 6 Gordon Yorke CLA 2011-06-15 09:14:23 EDT
Checked into revision 9306
Reviewed by Andrei Ilitchev
Comment 7 Eclipse Webmaster CLA 2022-06-09 10:04:04 EDT
The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink