From 5e986f5320c561953759a7beffb11db7e296817c Mon Sep 17 00:00:00 2001 From: Robert Varga Date: Tue, 27 Jun 2017 16:53:09 +0200 Subject: [PATCH] BUG-8494: fix failure path thinko The check should be to see if the failure has *not* been set, hence invert the check. Change-Id: I2c3893924f1c985687beedbfae0889388fad15c7 Signed-off-by: Robert Varga --- .../clustering/it/provider/impl/AbstractTransactionHandler.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/opendaylight/md-sal/samples/clustering-test-app/provider/src/main/java/org/opendaylight/controller/clustering/it/provider/impl/AbstractTransactionHandler.java b/opendaylight/md-sal/samples/clustering-test-app/provider/src/main/java/org/opendaylight/controller/clustering/it/provider/impl/AbstractTransactionHandler.java index f23e7ec194..250b3b30e9 100644 --- a/opendaylight/md-sal/samples/clustering-test-app/provider/src/main/java/org/opendaylight/controller/clustering/it/provider/impl/AbstractTransactionHandler.java +++ b/opendaylight/md-sal/samples/clustering-test-app/provider/src/main/java/org/opendaylight/controller/clustering/it/provider/impl/AbstractTransactionHandler.java @@ -53,7 +53,7 @@ abstract class AbstractTransactionHandler { @Override void txFailure(final ListenableFuture execFuture, final long txId, final Throwable cause) { futures.remove(execFuture); - if (failure != null) { + if (failure == null) { failure = cause; } } -- 2.36.6