wolfkill commented on issue #18218: URL: https://github.com/apache/pinot/issues/18218#issuecomment-4569952679
This appears to be covered by the merged #18405. The current `FilePerIndexDirectory.removeIndex()` closes the cached `PinotDataBuffer` before removing the map entry, and keeps the entry if close throws so `close()` can retry later. #18218 was probably left open because #18405 closed #18404 instead. Unless additional regression coverage is still desired here, this issue looks safe to close. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
