Yingyi Bu has submitted this change and it was merged. Change subject: Fix a mutual waiting condition during switching the txn log file: ......................................................................
Fix a mutual waiting condition during switching the txn log file: -- LogManager waits for LogBuffer to flush, -- LogBuffer is not full and cannot flush. Change-Id: I7bf7b70675ab68005e581a296ac4f4515f1d5ad4 Reviewed-on: https://asterix-gerrit.ics.uci.edu/1333 Sonar-Qube: Jenkins <[email protected]> Tested-by: Jenkins <[email protected]> Integration-Tests: Jenkins <[email protected]> Reviewed-by: Michael Blow <[email protected]> --- M asterixdb/asterix-transactions/src/main/java/org/apache/asterix/transaction/management/service/logging/LogManager.java 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Michael Blow: Looks good to me, approved Jenkins: Verified; No violations found; Verified 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 947ebc7..814bcfc 100644 --- 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 @@ -203,6 +203,8 @@ } protected void prepareNextLogFile() { + // Make sure to flush whatever left in the log tail. + appendPage.isFull(true); //wait until all log records have been flushed in the current file synchronized (flushLSN) { try { -- To view, visit https://asterix-gerrit.ics.uci.edu/1333 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7bf7b70675ab68005e581a296ac4f4515f1d5ad4 Gerrit-PatchSet: 3 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Yingyi Bu <[email protected]> Gerrit-Reviewer: Ian Maxon <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Michael Blow <[email protected]> Gerrit-Reviewer: Murtadha Hubail <[email protected]> Gerrit-Reviewer: Till Westmann <[email protected]> Gerrit-Reviewer: Yingyi Bu <[email protected]> Gerrit-Reviewer: abdullah alamoudi <[email protected]>
