Jackie-Jiang opened a new pull request #5400: URL: https://github.com/apache/incubator-pinot/pull/5400
The PinotDataBuffers are tracked and maintained inside SegmentDirectory for ImmutableSegment and PinotDataBufferMemoryManager for MutableSegment. They are created when loading the indexes and released when closing the segment. If the PinotDataBuffer gets released when closing the index, and if the buffer manager decide to reuse the buffer, the following read on the buffer will cause JVM to crash. This can be triggered in SegmentPreProcessor when the same indexes need to be opened twice in two different preprocessors. This PR standardize the behavior of indexes to not release (close) the PinotDataBuffer when closing the index. Also, after the index is closed, we should prevent reading anything from the PinotDataBuffer because the buffer might already be closed and it could cause JVM to crash. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
