This repository has been archived by the owner. It is now read-only.

Review use of caches #184

Closed
jodastephen opened this Issue Dec 20, 2012 · 2 comments

Comments

Projects
None yet
1 participant
@jodastephen
Member

jodastephen commented Dec 20, 2012

We need to find all the caches in JSR-310 and ensure that their use is acceptable and sensible. Caching needs in the JDK/JRE may also differ in the embedded version.

@jodastephen

This comment has been minimized.

Show comment
Hide comment
@jodastephen

jodastephen Apr 1, 2013

Member

We have caching as follows:

  • LocalTime holds a cache of each instance representing a whole hour
  • ZoneOffset holds a cache of all offset objects that are created (max cache size 129,601)
  • DateTimeTextProvider holds an cache of text by field/locale
  • Chronology holds a cache of chronologies
  • WeekFields holds a cache of instances
  • TzdbZoneRulesProvider and ZoneRulesProvider hold caches of time-zones
Member

jodastephen commented Apr 1, 2013

We have caching as follows:

  • LocalTime holds a cache of each instance representing a whole hour
  • ZoneOffset holds a cache of all offset objects that are created (max cache size 129,601)
  • DateTimeTextProvider holds an cache of text by field/locale
  • Chronology holds a cache of chronologies
  • WeekFields holds a cache of instances
  • TzdbZoneRulesProvider and ZoneRulesProvider hold caches of time-zones
@jodastephen

This comment has been minimized.

Show comment
Hide comment
@jodastephen

jodastephen Jul 4, 2013

Member

The only concerning one AFAICT was ZoneOffset, but that now caches only offsets such as xx:00, xx:15, xx:30 and xx:45, not all offsets.

Open a separate issue if there is a problem,

Member

jodastephen commented Jul 4, 2013

The only concerning one AFAICT was ZoneOffset, but that now caches only offsets such as xx:00, xx:15, xx:30 and xx:45, not all offsets.

Open a separate issue if there is a problem,

@jodastephen jodastephen closed this Jul 4, 2013

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.