siddharthteotia opened a new pull request #4882: Failure to recover after segment reload failure does not release segment lock URL: https://github.com/apache/incubator-pinot/pull/4882 This was noticed while going through some recent segment reload failure logs. While the best way to do this would be to use AutoCloseable but Lock interface is not AutoCloseable. An alternative would be to simply wrap it around AutoCloseable and then use try with resource but that will result in an unnecessary short-lived object. So right now, the fix is to just use additional 'finally' block.
---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
