package org.opendaylight.controller.clustering.it.provider.impl;
+import static org.opendaylight.controller.clustering.it.provider.impl.AbstractTransactionHandler.ITEM;
+
import com.google.common.base.Preconditions;
import com.google.common.util.concurrent.SettableFuture;
+import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
import java.util.Collection;
+import java.util.HashMap;
+import java.util.Map;
import java.util.concurrent.Executors;
import java.util.concurrent.Future;
import java.util.concurrent.ScheduledExecutorService;
import java.util.concurrent.ScheduledFuture;
import java.util.concurrent.TimeUnit;
+import java.util.concurrent.atomic.AtomicBoolean;
import java.util.concurrent.atomic.AtomicLong;
import javax.annotation.Nonnull;
import org.opendaylight.controller.md.sal.dom.api.ClusteredDOMDataTreeChangeListener;
+import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier;
+import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier.NodeIdentifierWithPredicates;
+import org.opendaylight.yangtools.yang.data.api.schema.MapEntryNode;
+import org.opendaylight.yangtools.yang.data.api.schema.MapNode;
import org.opendaylight.yangtools.yang.data.api.schema.NormalizedNode;
import org.opendaylight.yangtools.yang.data.api.schema.tree.DataTreeCandidate;
import org.slf4j.Logger;
private static final Logger LOG = LoggerFactory.getLogger(IdIntsListener.class);
private static final long SECOND_AS_NANO = 1000000000;
- private NormalizedNode<?, ?> localCopy = null;
+ private volatile NormalizedNode<?, ?> localCopy;
private final AtomicLong lastNotifTimestamp = new AtomicLong(0);
private ScheduledExecutorService executorService;
private ScheduledFuture<?> scheduledFuture;
+ private final AtomicBoolean loggedIgnoredNotificationDiff = new AtomicBoolean();
@Override
public void onDataTreeChanged(@Nonnull final Collection<DataTreeCandidate> changes) {
if (localCopy == null || checkEqual(change.getRootNode().getDataBefore().get())) {
localCopy = change.getRootNode().getDataAfter().get();
} else {
- LOG.warn("Ignoring notification.");
+ LOG.warn("Ignoring notification {}", loggedIgnoredNotificationDiff.compareAndSet(false, true)
+ ? diffWithLocalCopy(change.getRootNode().getDataBefore().get()) : "");
LOG.trace("Ignored notification content: {}", change);
}
} else {
return localCopy.equals(expected);
}
+ @SuppressFBWarnings("BC_UNCONFIRMED_CAST")
+ public String diffWithLocalCopy(final NormalizedNode<?, ?> expected) {
+ return diffNodes((MapNode)expected, (MapNode)localCopy);
+ }
+
public Future<Void> tryFinishProcessing() {
executorService = Executors.newSingleThreadScheduledExecutor();
final SettableFuture<Void> settableFuture = SettableFuture.create();
return settableFuture;
}
+ public static String diffNodes(final MapNode expected, final MapNode actual) {
+ StringBuilder builder = new StringBuilder("MapNodes diff:");
+
+ final YangInstanceIdentifier.NodeIdentifier itemNodeId = new YangInstanceIdentifier.NodeIdentifier(ITEM);
+
+ Map<NodeIdentifierWithPredicates, MapEntryNode> expIdIntMap = new HashMap<>();
+ expected.getValue().forEach(node -> expIdIntMap.put(node.getIdentifier(), node));
+
+ actual.getValue().forEach(actIdInt -> {
+ final MapEntryNode expIdInt = expIdIntMap.remove(actIdInt.getIdentifier());
+ if (expIdInt == null) {
+ builder.append('\n').append(" Unexpected id-int entry for ").append(actIdInt.getIdentifier());
+ return;
+ }
+
+ Map<NodeIdentifierWithPredicates, MapEntryNode> expItemMap = new HashMap<>();
+ ((MapNode)expIdInt.getChild(itemNodeId).get()).getValue()
+ .forEach(node -> expItemMap.put(node.getIdentifier(), node));
+
+ ((MapNode)actIdInt.getChild(itemNodeId).get()).getValue().forEach(actItem -> {
+ final MapEntryNode expItem = expItemMap.remove(actItem.getIdentifier());
+ if (expItem == null) {
+ builder.append('\n').append(" Unexpected item entry ").append(actItem.getIdentifier())
+ .append(" for id-int entry ").append(actIdInt.getIdentifier());
+ }
+ });
+
+ expItemMap.values().forEach(node -> builder.append('\n')
+ .append(" Actual is missing item entry ").append(node.getIdentifier())
+ .append(" for id-int entry ").append(actIdInt.getIdentifier()));
+ });
+
+ expIdIntMap.values().forEach(node -> builder.append('\n')
+ .append(" Actual is missing id-int entry for ").append(node.getIdentifier()));
+
+ return builder.toString();
+ }
+
private class CheckFinishedTask implements Runnable {
private final SettableFuture<Void> future;