You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2020/09/04 13:05:08 UTC

[GitHub] [druid] a2l007 edited a comment on pull request #10341: Fix result-level caching

a2l007 edited a comment 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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org