dsmiley commented on PR #1854: URL: https://github.com/apache/solr/pull/1854#issuecomment-1686750517
> I don't think it's a very good idea to mix functionality like this (metrics collection with tracing headers) I disagree; both are "instrumentation" and the class name was already chosen in this generic way instead of being called MetricsBlahBlahBlah. Nonetheless the concerns could be split up to work in parallel but it's extra code for unclear gain. BTW again in my fork locally, I did similarly for the Jetty side in InstrumentedHttpListenerFactory moving the previous logic to this central location, and I think in so doing picked up more use-cases. -- 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: issues-unsubscr...@solr.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For additional commands, e-mail: issues-h...@solr.apache.org
