Naming convention

Change-Id: I2e2bbce6397dd95e841bcdc67ee125c1626671b3

Naming convention
Change-Id: I2e2bbce6397dd95e841bcdc67ee125c1626671b3
Shashikanth VH · Gerrit Code Review
Commit 013a7bc06c576201acec372556dfb1ab855bac4f 013a7bc0 1 parent c7f49f94
Showing 2 changed files with 15 additions and 15 deletions
apps/segmentrouting/src/main/java/org/onosproject/segmentrouting/DefaultRoutingHandler.java
apps/segmentrouting/src/main/java/org/onosproject/segmentrouting/ECMPShortestPathGraph.java → apps/segmentrouting/src/main/java/org/onosproject/segmentrouting/EcmpShortestPathGraph.java
--- a/apps/segmentrouting/src/main/java/org/onosproject/segmentrouting/DefaultRoutingHandler.java
View file @013a7bc
+++ b/apps/segmentrouting/src/main/java/org/onosproject/segmentrouting/DefaultRoutingHandler.java
View file @013a7bc
@@ -44,8 +44,8 @@ public class DefaultRoutingHandler {
     private SegmentRoutingManager srManager;
     private RoutingRulePopulator rulePopulator;
-    private HashMap<DeviceId, ECMPShortestPathGraph> currentEcmpSpgMap;
+    private HashMap<DeviceId, EcmpShortestPathGraph> currentEcmpSpgMap;
-    private HashMap<DeviceId, ECMPShortestPathGraph> updatedEcmpSpgMap;
+    private HashMap<DeviceId, EcmpShortestPathGraph> updatedEcmpSpgMap;
     private DeviceConfiguration config;
     private final Lock statusLock = new ReentrantLock();
     private volatile Status populationStatus;
@@ -103,7 +103,7 @@ public class DefaultRoutingHandler {
                     continue;
                 }
-                ECMPShortestPathGraph ecmpSpg = new ECMPShortestPathGraph(sw.id(), srManager);
+                EcmpShortestPathGraph ecmpSpg = new EcmpShortestPathGraph(sw.id(), srManager);
                 if (!populateEcmpRoutingRules(sw.id(), ecmpSpg)) {
                     log.debug("populateAllRoutingRules: populationStatus is ABORTED");
                     populationStatus = Status.ABORTED;
@@ -149,8 +149,8 @@ public class DefaultRoutingHandler {
                 if (!srManager.mastershipService.isLocalMaster(sw.id())) {
                     continue;
                 }
-                ECMPShortestPathGraph ecmpSpgUpdated =
+                EcmpShortestPathGraph ecmpSpgUpdated =
-                        new ECMPShortestPathGraph(sw.id(), srManager);
+                        new EcmpShortestPathGraph(sw.id(), srManager);
                 updatedEcmpSpgMap.put(sw.id(), ecmpSpgUpdated);
             }
@@ -200,7 +200,7 @@ public class DefaultRoutingHandler {
             // When only the source device is defined, reinstall routes to all other devices
             if (link.size() == 1) {
                 log.trace("repopulateRoutingRulesForRoutes: running ECMP graph for device {}", link.get(0));
-                ECMPShortestPathGraph ecmpSpg = new ECMPShortestPathGraph(link.get(0), srManager);
+                EcmpShortestPathGraph ecmpSpg = new EcmpShortestPathGraph(link.get(0), srManager);
                 if (populateEcmpRoutingRules(link.get(0), ecmpSpg)) {
                     log.debug("Populating flow rules from {} to all is successful",
                               link.get(0));
@@ -228,7 +228,7 @@ public class DefaultRoutingHandler {
                           link.get(0), link.get(1));
                 DeviceId src = link.get(0);
                 DeviceId dst = link.get(1);
-                ECMPShortestPathGraph ecmpSpg = updatedEcmpSpgMap.get(dst);
+                EcmpShortestPathGraph ecmpSpg = updatedEcmpSpgMap.get(dst);
                 HashMap<Integer, HashMap<DeviceId, ArrayList<ArrayList<DeviceId>>>> switchVia =
                         ecmpSpg.getAllLearnedSwitchesAndVia();
                 for (Integer itrIdx : switchVia.keySet()) {
@@ -275,7 +275,7 @@ public class DefaultRoutingHandler {
             if (!srManager.mastershipService.isLocalMaster(sw.id())) {
                 continue;
             }
-            ECMPShortestPathGraph ecmpSpg = currentEcmpSpgMap.get(sw.id());
+            EcmpShortestPathGraph ecmpSpg = currentEcmpSpgMap.get(sw.id());
             if (ecmpSpg == null) {
                 log.error("No existing ECMP graph for switch {}", sw.id());
                 continue;
@@ -330,7 +330,7 @@ public class DefaultRoutingHandler {
             }
             log.debug("Checking route change for switch {}", sw.id());
-            ECMPShortestPathGraph ecmpSpg = currentEcmpSpgMap.get(sw.id());
+            EcmpShortestPathGraph ecmpSpg = currentEcmpSpgMap.get(sw.id());
             if (ecmpSpg == null) {
                 log.debug("No existing ECMP graph for device {}", sw.id());
                 ArrayList<DeviceId> route = new ArrayList<>();
@@ -338,7 +338,7 @@ public class DefaultRoutingHandler {
                 routes.add(route);
                 continue;
             }
-            ECMPShortestPathGraph newEcmpSpg = updatedEcmpSpgMap.get(sw.id());
+            EcmpShortestPathGraph newEcmpSpg = updatedEcmpSpgMap.get(sw.id());
             //currentEcmpSpgMap.put(sw.id(), newEcmpSpg);
             HashMap<Integer, HashMap<DeviceId, ArrayList<ArrayList<DeviceId>>>> switchVia =
                     ecmpSpg.getAllLearnedSwitchesAndVia();
@@ -414,7 +414,7 @@ public class DefaultRoutingHandler {
     }
     private boolean populateEcmpRoutingRules(DeviceId destSw,
-                                             ECMPShortestPathGraph ecmpSPG) {
+                                             EcmpShortestPathGraph ecmpSPG) {
         HashMap<Integer, HashMap<DeviceId, ArrayList<ArrayList<DeviceId>>>> switchVia = ecmpSPG
                 .getAllLearnedSwitchesAndVia();
--- a/apps/segmentrouting/src/main/java/org/onosproject/segmentrouting/ECMPShortestPathGraph.java → apps/segmentrouting/src/main/java/org/onosproject/segmentrouting/EcmpShortestPathGraph.java
View file @013a7bc
+++ b/apps/segmentrouting/src/main/java/org/onosproject/segmentrouting/ECMPShortestPathGraph.java → apps/segmentrouting/src/main/java/org/onosproject/segmentrouting/EcmpShortestPathGraph.java
View file @013a7bc
@@ -36,7 +36,7 @@ import java.util.List;
  * calculated. The paths are snapshot paths at the point of the class
  * instantiation.
  */
-public class ECMPShortestPathGraph {
+public class EcmpShortestPathGraph {
     LinkedList<DeviceId> deviceQueue = new LinkedList<>();
     LinkedList<Integer> distanceQueue = new LinkedList<>();
     HashMap<DeviceId, Integer> deviceSearched = new HashMap<>();
@@ -46,7 +46,7 @@ public class ECMPShortestPathGraph {
     DeviceId rootDevice;
     private SegmentRoutingManager srManager;
     private static final Logger log = LoggerFactory
-            .getLogger(ECMPShortestPathGraph.class);
+            .getLogger(EcmpShortestPathGraph.class);
     /**
      * Constructor.
@@ -55,7 +55,7 @@ public class ECMPShortestPathGraph {
      * @param linkListToAvoid link list to avoid
      * @param deviceIdListToAvoid device list to avoid
      */
-    public ECMPShortestPathGraph(DeviceId rootDevice, List<String> deviceIdListToAvoid,
+    public EcmpShortestPathGraph(DeviceId rootDevice, List<String> deviceIdListToAvoid,
                                  List<Link> linkListToAvoid) {
         this.rootDevice = rootDevice;
         calcECMPShortestPathGraph(deviceIdListToAvoid, linkListToAvoid);
@@ -67,7 +67,7 @@ public class ECMPShortestPathGraph {
      * @param rootDevice root of the BFS tree
      * @param srManager SegmentRoutingManager object
      */
-    public ECMPShortestPathGraph(DeviceId rootDevice, SegmentRoutingManager srManager) {
+    public EcmpShortestPathGraph(DeviceId rootDevice, SegmentRoutingManager srManager) {
         this.rootDevice = rootDevice;
         this.srManager = srManager;
         calcECMPShortestPathGraph();