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]

Reply via email to