On Tue, Oct 28, 2025 at 9:46 AM Matthew Biscocho (BLOOMBERG/ 919 3RD A) <
[email protected]> wrote:
> I am good with making changes for most of these recommendations and have a
> few thoughts on some of them.
>
> The solr_executor_ has an executor name attribute inside of it that should
> let you know it was associated with a core.
>
Nice to know but nonetheless these feel unorganized & inconsistent compared
to similar situations (like caches, handlers)
> For solr_searcher_cache_size vs solr_field_cache_size_bytes we could
> change the suffix for number of entries to solr_searcher_cache_entries to
> differentiate them better on the suffix.
>
+1
On "aggregateNodeLevelMetricsEnabled" I think I made a suggestion somewhere
> to just remove those base core attributes off the metric then the user
> could just assume that the metric is an aggregation like so:
>
> solr_core_requests_total{handler="/select",core="foo"} 5
> solr_core_requests_total{handler="/select",core="bar"} 5
> solr_core_requests_total{handler="/select"} 10 <- Node level aggregate
>
+1
