a2l007 commented on pull request #10341: URL: https://github.com/apache/druid/pull/10341#issuecomment-687131555
> I believe we need integration tests for this fix, but I'm not sure how to add it without introducing a metricsMonitor for integration tests. Curious, but are we trying to avoid introducing metricsMonitor to integration tests because it is outside the scope of this PR or are there other concerns with adding metricsMonitor to integration tests? ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
