Hello Anon. E. Moose #1000171, Jenkins, I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/2165 to look at the new patch set (#2). Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent ...................................................................... [NO ISSUE][TX] Make TxnLogFile Close Idempotent - user model changes: no - storage format changes: no - interface changes: yes Renamed ILogReader.initializeScan to setPosition and added javadocs. Details: Currently there is an explicit check that the file channel of a TxnLogFile is open before closing it. However, the channel could be closed due to interrupts and therefore we should remove the explicit check and always try to close it. However, we should always decrement the TxnLogFile references counter even if the channel is not open since that TxnLogFile is not accessed anymore. Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a --- M asterixdb/asterix-app/src/main/java/org/apache/asterix/app/nc/RecoveryManager.java M asterixdb/asterix-common/src/main/java/org/apache/asterix/common/transactions/ILogReader.java M asterixdb/asterix-transactions/src/main/java/org/apache/asterix/transaction/management/service/logging/LogManager.java M asterixdb/asterix-transactions/src/main/java/org/apache/asterix/transaction/management/service/logging/LogReader.java 4 files changed, 49 insertions(+), 45 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/65/2165/2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha Hubail <mhub...@apache.org> Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins <jenk...@fulliautomatix.ics.uci.edu> Gerrit-Reviewer: Michael Blow <mb...@apache.org> Gerrit-Reviewer: Till Westmann <ti...@apache.org> Gerrit-Reviewer: abdullah alamoudi <bamou...@gmail.com>