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

Bug 363385

Summary:

Performance Improvement - NamespaceResolver reduce unnecessary instances of Map

Product:

z_Archived

Reporter:

Blaise Doughan <blaise.doughan>

Component:

Eclipselink

Assignee:

Nobody - feel free to take it <nobody>

Status:

RESOLVED FIXED

QA Contact:

Severity:

normal

Priority:

CC:

eclipselink.oxm-inbox

Version:

unspecified

Target Milestone:

---

Hardware:

OS:

Windows XP

Whiteboard:

Attachments:

Description	Flags
Core - Fix	none
Core - Fix	none
Core - Fix	none
MOXy - Fix	none
Core - Fix (2.3.2)	none
Core - Fix (Trunk)	none

Description Blaise Doughan

2011-11-09 16:11:08 EST

Currently NamespaceResolver holds an instance of Map to maintain key/uri pairs even when it doesn't need one.

Comment 1 Blaise Doughan

2011-11-09 16:23:31 EST

Created attachment 206739 [details]
Core - Fix

Comment 2 Blaise Doughan

2011-11-11 13:57:10 EST

Created attachment 206866 [details]
Core - Fix

Comment 3 Blaise Doughan

2011-11-11 16:07:57 EST

Created attachment 206874 [details]
Core - Fix

Comment 4 Blaise Doughan

2011-11-11 16:08:38 EST

Created attachment 206875 [details]
MOXy - Fix

Comment 5 Blaise Doughan

2011-11-14 09:59:16 EST

Created attachment 206950 [details]
Core - Fix (2.3.2)

Comment 6 Blaise Doughan

2011-11-14 11:31:09 EST

Created attachment 206960 [details]
Core - Fix (Trunk)

Comment 7 Blaise Doughan

2011-11-14 13:29:14 EST

Fix checked into 2.3.2 at rev:  10383
Fix checked into trunk at rev:  10384

Code reviewed by:  Matt MacIvor

Fix Details:
Lazy initialize the prefixesToNamespaces map, and added a hasPrefixesToNamespaces check to avoid the lazy initialization when it is not required.

Comment 8 Eclipse Webmaster

2022-06-09 10:27:31 EDT

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