[Unicode]   Common Locale Data Repository : Bug Tracking Home | Site Map | Search

CLDR Ticket #8476(accepted)

Opened 4 years ago

Last modified 3 months ago

Change new XPathParts()...

Reported by: mark Owned by: mark
Component: perf Data Locale:
Phase: rc Review:
Weeks: Data Xpath:

Description (last modified by mark) (diff)

It is somewhat faster to call getInstance to get a frozen version (the frozen version can also prevent inadvertent errors. While it is probably not worth a concerted effort, people should probably use this when they are writing new code or touching old.

new XPathParts().set(path)


or if it allows modification,



Change History

comment:1 Changed 4 years ago by mark

  • Description modified (diff)

comment:2 Changed 4 years ago by mark

  • Description modified (diff)

comment:3 Changed 4 years ago by emmons

  • Status changed from new to accepted
  • Component changed from unknown to perf
  • Priority changed from assess to minor
  • Phase changed from dsub to rc
  • Milestone changed from UNSCH to 28
  • Owner changed from anybody to mark
  • type changed from unknown to tools

comment:4 Changed 3 years ago by mark

  • Priority changed from minor to major
  • Milestone changed from 28 to 29

Actually, I think this has the opportunity to be a significant performance win, because XPathParts.getFrozenInstance(path) drastically reduces memory thrash. Because we often have static or shared XPathParts, this can also reduce thread contention.

So changing to major, but 29.

Doing this could go in two phases.

  1. Replace all the set call sites by using XPathParts.getInstance(path);
  1. Progressively change files to use XPathParts.getFrozenInstance(path) instead; whenever that change fails in the tests, back out the changes.

comment:5 Changed 3 years ago by emmons

  • Milestone changed from 29 to upcoming

Auto move of all 29 -> upcoming

comment:6 Changed 11 months ago by mark

  • Priority changed from major to medium

comment:7 Changed 3 months ago by pedberg

  • Milestone changed from upcoming to UNSCH

CLDR 34 BRS closing item, move all upcoming → UNSCH


Add a comment

Modify Ticket

as accepted

E-mail address and user name can be saved in the Preferences.

Note: See TracTickets for help on using tickets.