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/06/18 16:31:54 UTC
[GitHub] [druid] suneet-s commented on pull request #10027: fix query memory leak
suneet-s commented on pull request #10027:
URL: https://github.com/apache/druid/pull/10027#issuecomment-646148028
@chenyuzhi459 There are more details on the code coverage requirements and how to run the tests locally here - https://github.com/apache/druid/blob/master/dev/code-review/code-coverage.md#running-code-coverage-locally
https://travis-ci.org/github/apache/druid/jobs/699729079 - this job is failing because line 104 in
BackgroundCachePopulator is untested. Maybe you can add a new test class `BackgroundCachePopulatorTest` that adds a test that ensures the onFailure method is called.
```
------------------------------------------------------------------------------
org/apache/druid/client/cache/BackgroundCachePopulator.java
------------------------------------------------------------------------------
27 import com.google.common.util.concurrent.ListenableFuture;
28 import com.google.common.util.concurrent.ListeningExecutorService;
29 import com.google.common.util.concurrent.MoreExecutors;
30 F import org.apache.druid.common.guava.GuavaUtils;
104 F | L GuavaUtils.cancelAll(cacheFutures);
------------------------------------------------------------------------------
```
I realize this test coverage was missing before your change, but it would help Druid raise the bar on testing if you are able to add a test for this.
Hope this helps.
----------------------------------------------------------------
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