364872 – Memory Improvement: DescriptorQueryManager empty ConcurrentHashMaps

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

Bug 364872 - Memory Improvement: DescriptorQueryManager empty ConcurrentHashMaps

Summary: Memory Improvement: DescriptorQueryManager empty ConcurrentHashMaps

Status:	RESOLVED FIXED

Alias:	None

Product:	z_Archived
Classification:	Eclipse Foundation
Component:	Eclipselink (show other bugs)
Version:	unspecified
Hardware:	PC Windows XP

Importance:	P3 normal (vote)
Target Milestone:	---
Assignee:	Nobody - feel free to take it
QA Contact:

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2011-11-25 15:56 EST by Rick Barkhouse
Modified:	2022-06-09 10:27 EDT (History)
CC List:	0 users

See Also:

Attachments
Proposed patch (3.77 KB, patch) 2011-11-25 15:58 EST, Rick Barkhouse	no flags	Details \| Diff
Proposed patch (3.77 KB, patch) 2011-11-25 15:59 EST, Rick Barkhouse	no flags	Details \| Diff
Proposed patch (3.89 KB, patch) 2011-12-16 13:43 EST, Rick Barkhouse	no flags	Details \| Diff
Show Obsolete (2) 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 Rick Barkhouse

2011-11-25 15:56:20 EST

DescriptorQueryManager holds two instances of ConcurrentFixedCache (which contains a ConcurrentHashMap), that could be lazily-initialized to improve memory usage.

In MOXy testing on a SpecJ model, these empty collections retained around 185k in memory.

Comment 1 Rick Barkhouse

2011-11-25 15:58:16 EST

Created attachment 207560 [details]
Proposed patch

Comment 2 Rick Barkhouse

2011-11-25 15:59:48 EST

Created attachment 207561 [details]
Proposed patch

Comment 3 Rick Barkhouse

2011-12-16 13:43:02 EST

Created attachment 208499 [details]
Proposed patch

Comment 4 Rick Barkhouse

2011-12-19 10:51:11 EST

Reviewed by James Sutherland and checked into main.

Comment 5 Eclipse Webmaster

2022-06-09 10:27:13 EDT

The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink