From 1c717bbf117d3486196a0fdd73ac650721f9c557 Mon Sep 17 00:00:00 2001 From: Michael Vorburger Date: Mon, 11 Dec 2017 23:24:39 +0100 Subject: [PATCH] ConcurrentDOMDataBroker LOG debug instead of error for "Tx: {} Error during phase {}, starting Abort" see CONTROLLER-1802 and NETVIRT-916 for the full background story to why this will help reduce confusion when we analyze logs. Change-Id: I7f791bc92d3c22d96462381d6b966755134647d4 Signed-off-by: Michael Vorburger --- .../controller/cluster/databroker/ConcurrentDOMDataBroker.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/databroker/ConcurrentDOMDataBroker.java b/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/databroker/ConcurrentDOMDataBroker.java index 6df71974f7..b80a6050fc 100644 --- a/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/databroker/ConcurrentDOMDataBroker.java +++ b/opendaylight/md-sal/sal-distributed-datastore/src/main/java/org/opendaylight/controller/cluster/databroker/ConcurrentDOMDataBroker.java @@ -204,7 +204,8 @@ public class ConcurrentDOMDataBroker extends AbstractDOMBroker { return; } - LOG.warn("Tx: {} Error during phase {}, starting Abort", transaction.getIdentifier(), phase, throwable); + // Use debug instead of warn level here because this exception gets propagate back to the caller via the Future + LOG.debug("Tx: {} Error during phase {}, starting Abort", transaction.getIdentifier(), phase, throwable); // Transaction failed - tell all cohorts to abort. @SuppressWarnings("unchecked") -- 2.36.6