commit 828bc16cac0ddea10c22a4fe93812806c7cba627
author Sho SHIMIZU <sshimizu@us.fujitsu.com> Wed Jan 13 23:10:43 2016 -0800
committer Gerrit Code Review <gerrit@onlab.us> Thu Jan 14 08:12:39 2016 +0000
tree 51241cb8089f9631480a97482f4bea28b632cb2c
parent 5df6518dd26d1305947cdd216d290d19500287be

Use JDK's Objects instead of Guava's Objects

Change-Id: I58e5013e953b3528a99c54712ac4f6a7f10b4cb4

core/store/dist/src/main/java/org/onosproject/store/flow/impl/NewDistributedFlowRuleStore.java

diff --git a/core/store/dist/src/main/java/org/onosproject/store/flow/impl/NewDistributedFlowRuleStore.java b/core/store/dist/src/main/java/org/onosproject/store/flow/impl/NewDistributedFlowRuleStore.java
index cc2f730..99dec1d 100644
--- a/core/store/dist/src/main/java/org/onosproject/store/flow/impl/NewDistributedFlowRuleStore.java
+++ b/core/store/dist/src/main/java/org/onosproject/store/flow/impl/NewDistributedFlowRuleStore.java
@@ -15,7 +15,6 @@
  */
 package org.onosproject.store.flow.impl;
 
- import com.google.common.base.Objects;
  import com.google.common.collect.ImmutableList;
  import com.google.common.collect.ImmutableMap;
  import com.google.common.collect.Iterables;
@@ -85,6 +84,7 @@
  import java.util.HashSet;
  import java.util.List;
  import java.util.Map;
+ import java.util.Objects;
  import java.util.Set;
  import java.util.concurrent.ExecutorService;
  import java.util.concurrent.Executors;
@@ -364,7 +364,7 @@
             return null;
         }
 
-        if (Objects.equal(local, master)) {
+        if (Objects.equals(local, master)) {
             return flowTable.getFlowEntry(rule);
         }
 
@@ -390,7 +390,7 @@
             return Collections.emptyList();
         }
 
-        if (Objects.equal(local, master)) {
+        if (Objects.equals(local, master)) {
             return flowTable.getFlowEntries(deviceId);
         }
 
@@ -437,7 +437,7 @@
             return;
         }
 
-        if (Objects.equal(local, master)) {
+        if (Objects.equals(local, master)) {
             storeBatchInternal(operation);
             return;
         }
@@ -512,7 +512,7 @@
                     }
                     return null;
                 }
-        ).filter(op -> op != null).collect(Collectors.toSet());
+        ).filter(Objects::nonNull).collect(Collectors.toSet());
     }
 
     @Override
@@ -528,7 +528,7 @@
     @Override
     public FlowRuleEvent addOrUpdateFlowRule(FlowEntry rule) {
         NodeId master = mastershipService.getMasterFor(rule.deviceId());
-        if (Objects.equal(local, master)) {
+        if (Objects.equals(local, master)) {
             return addOrUpdateFlowRuleInternal(rule);
         }
 
@@ -564,7 +564,7 @@
         final DeviceId deviceId = rule.deviceId();
         NodeId master = mastershipService.getMasterFor(deviceId);
 
-        if (Objects.equal(local, master)) {
+        if (Objects.equals(local, master)) {
             // bypass and handle it locally
             return removeFlowRuleInternal(rule);
         }
@@ -619,7 +619,7 @@
 
             final DeviceId deviceId = operation.deviceId();
             NodeId master = mastershipService.getMasterFor(deviceId);
-            if (!Objects.equal(local, master)) {
+            if (!Objects.equals(local, master)) {
                 Set<FlowRule> failures = new HashSet<>(operation.size());
                 for (FlowRuleBatchEntry op : operation.getOperations()) {
                     failures.add(op.target());
@@ -656,13 +656,13 @@
             if (event.type() == ReplicaInfoEvent.Type.BACKUPS_CHANGED) {
                 DeviceId deviceId = event.subject();
                 NodeId master = mastershipService.getMasterFor(deviceId);
-                if (!Objects.equal(local, master)) {
+                if (!Objects.equals(local, master)) {
                  // ignore since this event is for a device this node does not manage.
                     return;
                 }
                 NodeId newBackupNode = getBackupNode(deviceId);
                 NodeId currentBackupNode = lastBackupNodes.get(deviceId);
-                if (Objects.equal(newBackupNode, currentBackupNode)) {
+                if (Objects.equals(newBackupNode, currentBackupNode)) {
                     // ignore since backup location hasn't changed.
                     return;
                 }
@@ -834,7 +834,7 @@
                                 NodeId lastBackupNode = lastBackupNodes.get(deviceId);
                                 NodeId newBackupNode = getBackupNode(deviceId);
                                 return lastBackupTime == null
-                                        ||  !Objects.equal(lastBackupNode, newBackupNode)
+                                        ||  !Objects.equals(lastBackupNode, newBackupNode)
                                         || (lastUpdateTime != null && lastUpdateTime > lastBackupTime);
                             })
                             .collect(Collectors.toSet());
@@ -862,7 +862,7 @@
             try {
                 flowTables.forEach((deviceId, deviceFlowTable) -> {
                     // Only process those devices are that not managed by the local node.
-                    if (!Objects.equal(local, mastershipService.getMasterFor(deviceId))) {
+                    if (!Objects.equals(local, mastershipService.getMasterFor(deviceId))) {
                         Map<FlowId, Map<StoredFlowEntry, StoredFlowEntry>> backupFlowTable =
                                 getFlowTable(deviceId);
                         backupFlowTable.clear();