> What I meant for removing remaining codes writing data on ZK is [this kind of 
> things](https://github.com/apache/incubator-druid/blob/master/indexing-service/src/main/java/io/druid/indexing/overlord/hrtr/HttpRemoteTaskRunnerFactory.java#L54).
>  

I see, well that specific code snippet has to exist to allow users change from 
`RemoteTaskRunner` to `HttpRemoteTaskRunner` cleanly. MiddleManagers are 
configured to keep on announcing task status in ZK even after you switch 
overlord to use HTTP , so the zk cleanup still needs to happen. Once everyone 
is using HTTP remote task runner, then we can disable MM from announcing task 
status in ZK and remove the code you mentioned.

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

Reply via email to