commit 435f23c48de54473f12822cf7b4b9b4e2b2795b9
author Yuta HIGUCHI <y-higuchi@ak.jp.nec.com> Fri Mar 09 09:01:53 2018 -0800
committer Ray Milkey <ray@opennetworking.org> Mon Mar 12 21:12:16 2018 +0000
tree d47acae7b1f32012d07c497d7586e290fda71cb3
parent 576969a0ed29514c3366a90f9c4af30f51ff4cb5

Gracefully handle Device not there yet

- according to the javadoc, this should return
  DomainId or null

Change-Id: Idf1a1719bd987ed1d22e4801a7a2fa8a22fd532b

core/net/src/main/java/org/onosproject/net/domain/impl/DomainManager.java

diff --git a/core/net/src/main/java/org/onosproject/net/domain/impl/DomainManager.java b/core/net/src/main/java/org/onosproject/net/domain/impl/DomainManager.java
index 9a8a257..8f0db20 100644
--- a/core/net/src/main/java/org/onosproject/net/domain/impl/DomainManager.java
+++ b/core/net/src/main/java/org/onosproject/net/domain/impl/DomainManager.java
@@ -31,6 +31,7 @@
 import org.slf4j.LoggerFactory;
 
 import java.util.HashSet;
+import java.util.Optional;
 import java.util.Set;
 
 import static com.google.common.base.Preconditions.checkNotNull;
@@ -81,7 +82,9 @@
     @Override
     public DomainId getDomain(DeviceId deviceId) {
         checkNotNull(deviceId);
-        return checkNotNull(getAnnotatedDomainId(deviceService.getDevice(deviceId)));
+        return Optional.ofNullable(deviceService.getDevice(deviceId))
+                    .map(this::getAnnotatedDomainId)
+                    .orElse(null);
     }
 
     private DomainId getAnnotatedDomainId(Device device) {