bikash-barnwal commented on PR #43189: URL: https://github.com/apache/superset/pull/43189#issuecomment-5550394652
Fixed both: - Set `fontSize: theme.fontSizeSM` on the markLine label so it always matches what `measureTextWidth` assumed. Added an assertion pinning that in the existing grid-room test. - Capped `TEXT_WIDTH_CACHE` at 2000 entries with LRU eviction (re-insert on hit to track recency, evict the oldest on overflow). Added dedicated cache tests: hit/miss by cache key, and eviction of the least-recently-used entry once full. All tests pass locally (both files). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
