Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 351610 - Add "version" field to debug printing when an IdentityMap is printed
Summary: Add "version" field to debug printing when an IdentityMap is printed
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Eclipselink (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: David Minsky CLA
QA Contact: Project Inbox CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-07-08 17:24 EDT by David Minsky CLA
Modified: 2022-06-09 10:09 EDT (History)
0 users

See Also:

Attachments
Proposed patch (3.05 KB, patch)
2011-07-08 17:25 EDT, David Minsky CLA 		no flags Details | Diff
Proposed patch (v2) (3.12 KB, patch)
2011-07-14 14:34 EDT, David Minsky CLA 		no flags Details | Diff
Show Obsolete (1) 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 David Minsky CLA 2011-07-08 17:24:00 EDT 
Currently, when invoking session.getIdentityMapAccessor().printIdentityMaps(), the output prints (for each Descriptor) each object's id, identity hashcode, and the object itself (String representation of the object).

I propose that it would be useful and beneficial to debugging for each object's version (write lock) field to also be printed along with the existing information, if the ClassDescriptor is configured to use Optimistic Locking with version locking.

This would be an extension of the current debug printing code.
Comment 1 David Minsky CLA 2011-07-08 17:25:14 EDT 
Created attachment 199365 [details]
Proposed patch
Comment 2 David Minsky CLA 2011-07-14 14:34:27 EDT 
Created attachment 199692 [details]
Proposed patch (v2)
Comment 3 David Minsky CLA 2011-07-14 18:35:39 EDT 
Fixed in trunk (2.4) at revision: 9734
2.3.1 commit pending
Comment 4 David Minsky CLA 2011-07-15 17:20:08 EDT 
Checked into 2.3.1 at r9745
Comment 5 Eclipse Webmaster CLA 2022-06-09 10:09:23 EDT 
The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink