dsmiley commented on PR #4226:
URL: https://github.com/apache/solr/pull/4226#issuecomment-4139896222
Assuming tests pass etc., I plan to merge Saturday.
It's slightly tempting to leave `unregister` as an alias for close but...
this is internal.
Commit message:
----
SolrMetricsContext lifecycle management
- Made SolrMetricsContext implement AutoCloseable and renamed unregister()
→ close()
- Added ObjectReleaseTracker integration for leak detection
- Updated 30+ components to properly close their metrics contexts (caches,
handlers, security plugins, update components, etc.)
- Fixed double-close issues causing OTEL warnings
- Added assertions in RequestHandlerBase to enforce metrics initialization
before request handling
--
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]
