PathIntentInstaller: avoiding calls to Resource Store when unnecessary

Change-Id: I846ca543f922063ddc67a06353f43c590bd57d96

@@ -114,10 +114,8 @@ public class PathIntentInstaller implements IntentInstaller<PathIntent> {
 
 
     @Override
     @Override
     public List<FlowRuleBatchOperation> uninstall(PathIntent intent) {
     public List<FlowRuleBatchOperation> uninstall(PathIntent intent) {
-        LinkResourceAllocations allocatedResources = resourceService.getAllocations(intent.id());
-        if (allocatedResources != null) {
-            resourceService.releaseResources(allocatedResources);
-        }
+        deallocateResources(intent);
+
         TrafficSelector.Builder builder =
         TrafficSelector.Builder builder =
                 DefaultTrafficSelector.builder(intent.selector());
                 DefaultTrafficSelector.builder(intent.selector());
         Iterator<Link> links = intent.path().links().iterator();
         Iterator<Link> links = intent.path().links().iterator();
@@ -167,4 +165,20 @@ public class PathIntentInstaller implements IntentInstaller<PathIntent> {
         LinkResourceRequest request = builder.build();
         LinkResourceRequest request = builder.build();
         return request.resources().isEmpty() ? null : resourceService.requestResources(request);
         return request.resources().isEmpty() ? null : resourceService.requestResources(request);
     }
     }
+
+    /**
+     * Deallocate resources held by an intent.
+     *
+     * @param intent intent to deallocate resources for
+     */
+    private void deallocateResources(PathIntent intent) {
+        if (intent.constraints().isEmpty()) {
+            return;
+        }
+
+        LinkResourceAllocations allocatedResources = resourceService.getAllocations(intent.id());
+        if (allocatedResources != null) {
+            resourceService.releaseResources(allocatedResources);
+        }
+    }
 }
 }