Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Murtadha Hubail has submitted this change and it was merged. 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 Reviewed-on: https://asterix-gerrit.ics.uci.edu/2165 Reviewed-by: Till WestmannSonar-Qube: Jenkins Tested-by: Jenkins Integration-Tests: Jenkins --- 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(-) Approvals: Anon. E. Moose #1000171: No violations found Till Westmann: Looks good to me, approved Jenkins: Verified; No violations found; Verified diff --git a/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/nc/RecoveryManager.java b/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/nc/RecoveryManager.java index 7bc5697..19966fe 100644 --- a/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/nc/RecoveryManager.java +++ b/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/nc/RecoveryManager.java @@ -201,7 +201,7 @@ jobId2WinnerEntitiesMap = new HashMap<>(); //set log reader to the lowWaterMarkLsn ILogRecord logRecord; -logReader.initializeScan(lowWaterMarkLSN); +logReader.setPosition(lowWaterMarkLSN); logRecord = logReader.next(); while (logRecord != null) { if (IS_DEBUG_MODE) { @@ -300,7 +300,7 @@ ILogRecord logRecord = null; try { -logReader.initializeScan(lowWaterMarkLSN); +logReader.setPosition(lowWaterMarkLSN); logRecord = logReader.next(); while (logRecord != null) { if (IS_DEBUG_MODE) { @@ -540,7 +540,7 @@ Set activePartitions = localResourceRepository.getActivePartitions(); ILogReader logReader = logMgr.getLogReader(false); try { -logReader.initializeScan(firstLSN); +logReader.setPosition(firstLSN); ILogRecord logRecord = null; while (currentLSN < lastLSN) { logRecord = logReader.next(); diff --git a/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/transactions/ILogReader.java b/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/transactions/ILogReader.java index da188e3..8539e2b 100644 --- a/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/transactions/ILogReader.java +++ b/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/transactions/ILogReader.java @@ -18,18 +18,34 @@ */ package org.apache.asterix.common.transactions; -import org.apache.asterix.common.exceptions.ACIDException; - public interface ILogReader { -public void initializeScan(long beginLSN) throws ACIDException; +/** + * Sets the log reader position at log sequence number with value {@code lsn}. + * + * @param lsn + */ +void setPosition(long lsn); -//for scanning -public ILogRecord next() throws ACIDException; +/** + * Reads and returns the log record located at the log reader current position. After reading the log record, + * the log reader position is incremented by the size of the read log. + * + * @return the log record + */ +ILogRecord next(); -//for random reading -public ILogRecord read(long readLSN) throws ACIDException; +/** + * Reads and returns the log record with log sequence number {@code lsn}. + * + * @param lsn + * @return The log record + */ +ILogRecord read(long lsn); -public void close() throws ACIDException; +/** + * Closes the log reader and any resources used. + */ +void close(); -} +} \ No newline at end of file diff --git
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Anon. E. Moose #1000171 has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 2: Contrib+2 Analytics Compatibility Tests Successful Build 2999 : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 2: Integration-Tests+1 Integration Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/4673/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 2: -Contrib BAD Compatibility Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/2180/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Anon. E. Moose #1000171 has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 2: Contrib-2 Analytics Compatibility Tests Failed Build 2998 : UNSTABLE -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 2: BAD Compatibility Tests Started https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/2180/ -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 2: Contrib-2 BAD Compatibility Tests Failed https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/2179/ : UNSTABLE -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 2: BAD Compatibility Tests Started https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/2179/ -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 2: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/4673/ -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Till Westmann has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 2: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/2333/ (11/11) -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app/2522/ (10/11) -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Murtadha Hubail has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 1: (3 comments) https://asterix-gerrit.ics.uci.edu/#/c/2165/1/asterixdb/asterix-transactions/src/main/java/org/apache/asterix/transaction/management/service/logging/LogManager.java File asterixdb/asterix-transactions/src/main/java/org/apache/asterix/transaction/management/service/logging/LogManager.java: PS1, Line 582: : : > Should we warn here? Or do we need to consider this a "regular case"? It is a rare regular case. Logging it wouldn't do any harm and might help us in debugging one day. I will add the log. https://asterix-gerrit.ics.uci.edu/#/c/2165/1/asterixdb/asterix-transactions/src/main/java/org/apache/asterix/transaction/management/service/logging/LogReader.java File asterixdb/asterix-transactions/src/main/java/org/apache/asterix/transaction/management/service/logging/LogReader.java: PS1, Line 150: ACIDException > What happens if we stop here instead of ignoring the interrupt and continui This would be a txn abort thread waiting for the flushLSN. If such thread is interrupt, it means that we are shutting down the NC, therefore, when the NC starts up, recovery will redo the committed records and there wont be anything to undo. I think this is better than leaving the thread waiting during shutdown. PS1, Line 288: logFile.close() > Should we set logFile to null here or do we still need it? This is assuming that the caller would adhere to the contract of ILogReader (i.e. close will be called when the logReader will no longer be used. However, calling close twice on the same log file has no effect since there is protection against that in TxnLogFile#close. It is similar to calling close multiple times on a file channel. Setting it to null wouldn't harm too. Done. -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Murtadha Hubail Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: Yes
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/1661/ (1/11) -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/2269/ (7/11) -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-assemblies/1976/ (8/11) -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/1654/ (2/11) -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-rebalance-cancellation/841/ (9/11) -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/7194/ (6/11) -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/1642/ (5/11) -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/5696/ (4/11) -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/2312/ (3/11) -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
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 HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Till Westmann has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 1: (3 comments) https://asterix-gerrit.ics.uci.edu/#/c/2165/1/asterixdb/asterix-transactions/src/main/java/org/apache/asterix/transaction/management/service/logging/LogManager.java File asterixdb/asterix-transactions/src/main/java/org/apache/asterix/transaction/management/service/logging/LogManager.java: PS1, Line 582: : : Should we warn here? Or do we need to consider this a "regular case"? https://asterix-gerrit.ics.uci.edu/#/c/2165/1/asterixdb/asterix-transactions/src/main/java/org/apache/asterix/transaction/management/service/logging/LogReader.java File asterixdb/asterix-transactions/src/main/java/org/apache/asterix/transaction/management/service/logging/LogReader.java: PS1, Line 150: ACIDException What happens if we stop here instead of ignoring the interrupt and continuing until we reach the flushLSN? PS1, Line 288: logFile.close() Should we set logFile to null here or do we still need it? -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Michael Blow Gerrit-Reviewer: Till Westmann Gerrit-Reviewer: abdullah alamoudi Gerrit-HasComments: Yes
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Anon. E. Moose #1000171 has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 1: Contrib+2 Analytics Compatibility Tests Successful Build 2993 : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 1: Integration-Tests+1 Integration Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/4659/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 1: Contrib+2 BAD Compatibility Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/2174/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 1: BAD Compatibility Tests Started https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/2174/ -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 1: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/4659/ -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/7180/ (3/11) -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/2319/ (4/11) -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-assemblies/1962/ (9/11) -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/5682/ (11/11) -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/2297/ (6/11) -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/1640/ (7/11) -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app/2508/ (2/11) -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/1628/ (5/11) -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-rebalance-cancellation/827/ (8/11) -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/2255/ (1/11) -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Murtadha Hubail has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/2165 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, 47 insertions(+), 47 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/65/2165/1 diff --git a/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/nc/RecoveryManager.java b/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/nc/RecoveryManager.java index 7bc5697..19966fe 100644 --- a/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/nc/RecoveryManager.java +++ b/asterixdb/asterix-app/src/main/java/org/apache/asterix/app/nc/RecoveryManager.java @@ -201,7 +201,7 @@ jobId2WinnerEntitiesMap = new HashMap<>(); //set log reader to the lowWaterMarkLsn ILogRecord logRecord; -logReader.initializeScan(lowWaterMarkLSN); +logReader.setPosition(lowWaterMarkLSN); logRecord = logReader.next(); while (logRecord != null) { if (IS_DEBUG_MODE) { @@ -300,7 +300,7 @@ ILogRecord logRecord = null; try { -logReader.initializeScan(lowWaterMarkLSN); +logReader.setPosition(lowWaterMarkLSN); logRecord = logReader.next(); while (logRecord != null) { if (IS_DEBUG_MODE) { @@ -540,7 +540,7 @@ Set activePartitions = localResourceRepository.getActivePartitions(); ILogReader logReader = logMgr.getLogReader(false); try { -logReader.initializeScan(firstLSN); +logReader.setPosition(firstLSN); ILogRecord logRecord = null; while (currentLSN < lastLSN) { logRecord = logReader.next(); diff --git a/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/transactions/ILogReader.java b/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/transactions/ILogReader.java index da188e3..8539e2b 100644 --- a/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/transactions/ILogReader.java +++ b/asterixdb/asterix-common/src/main/java/org/apache/asterix/common/transactions/ILogReader.java @@ -18,18 +18,34 @@ */ package org.apache.asterix.common.transactions; -import org.apache.asterix.common.exceptions.ACIDException; - public interface ILogReader { -public void initializeScan(long beginLSN) throws ACIDException; +/** + * Sets the log reader position at log sequence number with value {@code lsn}. + * + * @param lsn + */ +void setPosition(long lsn); -//for scanning -public ILogRecord next() throws ACIDException; +/** + * Reads and returns the log record located at the log reader current position. After reading the log record, + * the log reader position is incremented by the size of the read log. + * + * @return the log record + */ +ILogRecord next(); -//for random reading -public ILogRecord read(long readLSN) throws ACIDException; +/** + * Reads and returns the log record with log sequence number {@code lsn}. + * + * @param lsn + * @return The log record + */ +ILogRecord read(long lsn); -public void close() throws ACIDException; +/** + * Closes the log reader and any resources used. + */ +void close(); -} +} \ No newline at end of file diff --git a/asterixdb/asterix-transactions/src/main/java/org/apache/asterix/transaction/management/service/logging/LogManager.java b/asterixdb/asterix-transactions/src/main/java/org/apache/asterix/transaction/management/service/logging/LogManager.java index dd0a5c7..1cf7a50 100644 --- a/asterixdb/asterix-transactions/src/main/java/org/apache/asterix/transaction/management/service/logging/LogManager.java +++
Change in asterixdb[master]: [NO ISSUE][TX] Make TxnLogFile Close Idempotent
Jenkins has posted comments on this change. Change subject: [NO ISSUE][TX] Make TxnLogFile Close Idempotent .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/1647/ (10/11) -- To view, visit https://asterix-gerrit.ics.uci.edu/2165 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I255e4b9af0bc78298c0a25daf0b5629d413eba6a Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Murtadha HubailGerrit-Reviewer: Jenkins Gerrit-HasComments: No