diff options
author | Martin Polden <mpolden@mpolden.no> | 2020-11-03 09:00:20 +0100 |
---|---|---|
committer | Martin Polden <mpolden@mpolden.no> | 2020-11-03 09:00:20 +0100 |
commit | cfc8c69a70a153281002db7dad0af05cbddc1fc5 (patch) | |
tree | f9e39c679e23b84d0f79f9e29c510182f1b92f97 /config-provisioning | |
parent | fe688d87e4ff96dc63d797eaba5582d2594d7076 (diff) |
Do not commit and unlock together
You always want to unlock, but not always commit.
Diffstat (limited to 'config-provisioning')
-rw-r--r-- | config-provisioning/src/main/java/com/yahoo/config/provision/ApplicationTransaction.java | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/config-provisioning/src/main/java/com/yahoo/config/provision/ApplicationTransaction.java b/config-provisioning/src/main/java/com/yahoo/config/provision/ApplicationTransaction.java index d21f7cfe5ae..327dfd408b8 100644 --- a/config-provisioning/src/main/java/com/yahoo/config/provision/ApplicationTransaction.java +++ b/config-provisioning/src/main/java/com/yahoo/config/provision/ApplicationTransaction.java @@ -1,7 +1,6 @@ // Copyright Verizon Media. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root. package com.yahoo.config.provision; -import com.yahoo.transaction.Mutex; import com.yahoo.transaction.NestedTransaction; import java.io.Closeable; @@ -33,12 +32,7 @@ public class ApplicationTransaction implements Closeable { @Override public void close() { - try { - transaction.commit(); - } - finally { - lock.close(); - } + lock.close(); } } |