One of the things I was wondering is if the change could be simplified by maintaining `indicesOpen` state at the `LogSegment` layer. The first operation that needs to read from or write into any one of the indices could then trigger all of them to be opened. We could probably keep all the logic out of the index layers this way and instead have it all be in one place, in `LogSegment`.
I didn't spend too much time thinking through how much change something like this would require but I wanted to bring it up anyway. Just a suggestion to see if the changes can be simplified - the patch looks good to me in its current form as well. [ Full content available at: https://github.com/apache/kafka/pull/5498 ] This message was relayed via gitbox.apache.org for [email protected]
