Reuse attributes after apply import policy

Update attributes only if there has been any
change applied by import policy. Otherwise
attibutes are the same for the route that
has been already stored.

Change-Id: I288cea16badcd81ce4d5aef21214394421bb992c
Signed-off-by: Claudio D. Gasparini <claudio.gasparini@pantheon.tech>

diff --git a/bgp/rib-impl/src/main/java/org/opendaylight/protocol/bgp/rib/impl/EffectiveRibInWriter.java b/bgp/rib-impl/src/main/java/org/opendaylight/protocol/bgp/rib/impl/EffectiveRibInWriter.java
index 1b46203fdf61e87f1e85d3641d727bf1fc0c31c8..4b0bde1fdb1629d9a461a2c78d9406da5cb6ae1b 100644 (file)
--- a/bgp/rib-impl/src/main/java/org/opendaylight/protocol/bgp/rib/impl/EffectiveRibInWriter.java
+++ b/bgp/rib-impl/src/main/java/org/opendaylight/protocol/bgp/rib/impl/EffectiveRibInWriter.java
@@ -395,8 +395,11 @@ final class EffectiveRibInWriter implements PrefixesReceivedCounters, PrefixesIn
                 CountersUtil.increment(this.prefixesInstalled.get(tablesKey), tablesKey);
 
                 final YangInstanceIdentifier attPath = routePath.node(ribSupport.routeAttributesIdentifier());
-                final ContainerNode finalAttribute = ribSupport.attributeToContainerNode(attPath, optEffAtt.get());
-                tx.put(LogicalDatastoreType.OPERATIONAL, attPath, finalAttribute);
+                final Attributes attToStore = optEffAtt.get();
+                if(!attToStore.equals(routeAttrs)) {
+                    final ContainerNode finalAttribute = ribSupport.attributeToContainerNode(attPath, attToStore);
+                    tx.put(LogicalDatastoreType.OPERATIONAL, attPath, finalAttribute);
+                }
                 break;
             default:
                 LOG.warn("Ignoring unhandled route {}", route);