It is used in the tests:
```
val uncleanablePartitions =
cleaner.cleanerManager.uncleanablePartitions(uncleanableDirectory)
assertTrue(uncleanablePartitions.contains(topicPartitions(0)))
assertTrue(uncleanablePartitions.contains(topicPartitions(1)))
assertFalse(uncleanablePartitions.contains(topicPartitions(2)))
```
We could either
a) keep the method, currently for test use only
b) remove the method and remove the assertion
c) remove the method and make the `uncleanablePartitions` field public so the
test can access it
I've gone with b). Do confirm if you think this is the best approach
[ Full content available at: https://github.com/apache/kafka/pull/5439 ]
This message was relayed via gitbox.apache.org for [email protected]