@leventov thanks. I've also noticed that before. The problem is that the 
supervisor periodically executes `runNotice` and this can potentially call 
every method of the supervisor which in turn requires to mock everything using 
`EasyMock`. 

I think it's better to refactor the whole KafkaSupervisorTest to be based on a 
sort of more controllable mockup environment by creating `taskRunner`, 
`taskMaster`, `taskStorage`, and `taskClient` for test purpose instead of 
making them using EasyMock.

Raised https://github.com/apache/incubator-druid/issues/6296.

[ Full content available at: 
https://github.com/apache/incubator-druid/pull/6207 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to