[LUCENE-5303] OrdinalsCache should use reader.getCoreCacheKey()

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 4.6, 6.0
• Component/s: modules/facet
• Lucene Fields: New, Patch Available
• Assignee: Shai Erera
• Reporter: Michael McCandless
• Created: 23/Oct/13 19:20
• Updated: 09/May/16 18:35
• Resolved: 23/Oct/13 20:45

Description

I'm doing some facet performance tests, and I tried using the CachedOrdsCountingFacetsAggregator to cache the decoded ords per doc X field ... but noticed it was generating way too many cache entries, because it's currently using the NDV instance as the cache key.

NDV instances are thread-private so this results in way too many entries in the cache.