bug fixes and cosmetic changes about OpticalPathProvisioner:
- removeSink() at deactivate
- added log messages
- wrapped null return value with Optional
Change-Id: Ic5361df3cf82f99af315b2551d33bb84aed241ed
diff --git a/apps/newoptical/src/main/java/org/onosproject/newoptical/api/OpticalPathService.java b/apps/newoptical/src/main/java/org/onosproject/newoptical/api/OpticalPathService.java
index e6df0cb..9a31127 100644
--- a/apps/newoptical/src/main/java/org/onosproject/newoptical/api/OpticalPathService.java
+++ b/apps/newoptical/src/main/java/org/onosproject/newoptical/api/OpticalPathService.java
@@ -24,6 +24,7 @@
import java.time.Duration;
import java.util.List;
+import java.util.Optional;
/**
* Service to setup optical domain connectivity.
@@ -64,7 +65,7 @@
/**
* Returns path assigned to given ID.
* @param id ID of connectivity
- * @return list of link that compose a path. null if ID is invalid.
+ * @return list of link that compose a path. empty if ID is invalid.
*/
- List<Link> getPath(OpticalConnectivityId id);
+ Optional<List<Link>> getPath(OpticalConnectivityId id);
}