I'm not sure; this is the root of my general un-easiness with this change.

If I understand the API correctly, when we release the lock and close the 
channel, we are removing the lockfile. If we get an IOException, it *might* be 
because the file is already gone, but it might also be any number of other 
causes.

If we stop tracking the lock before we are sure the lockfile is gone, don't we 
risk a kind of deadlock in which the lockfile is there, but no one thinks they 
own it, and therefore no one will release it?

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

Reply via email to