bowsii commented on PR #4708: URL: https://github.com/apache/polaris/pull/4708#issuecomment-4688025561
Thanks for the suggestion! I've taken it one step further and removed `MetricsPersistence` from `PolarisCallContext` entirely. The request-scoped producer now resolves it directly from `MetaStoreManagerFactory`, following the same logic previously used during call context creation. This means `MetricsPersistence` is only materialized for requests that actually inject it (such as metrics reporting), while non-metrics API calls no longer create or carry it in the call context. I've also updated the affected call sites and tests accordingly. -- 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]
