Dist.FlowRuleStore: return empty when no master
Change-Id: I377003b5e1631ead05ab3d32bf670f52372de55d
Showing
1 changed file
with
3 additions
and
2 deletions
... | @@ -284,9 +284,10 @@ public class DistributedFlowRuleStore | ... | @@ -284,9 +284,10 @@ public class DistributedFlowRuleStore |
284 | 284 | ||
285 | if (!replicaInfo.master().isPresent()) { | 285 | if (!replicaInfo.master().isPresent()) { |
286 | log.warn("No master for {}", deviceId); | 286 | log.warn("No master for {}", deviceId); |
287 | - // TODO: revisit if this should be returning empty collection. | 287 | + // TODO: revisit if this should be returning empty collection or throwing exception. |
288 | // FIXME: throw a FlowStoreException | 288 | // FIXME: throw a FlowStoreException |
289 | - throw new RuntimeException("No master for " + deviceId); | 289 | + //throw new RuntimeException("No master for " + deviceId); |
290 | + return Collections.emptyList(); | ||
290 | } | 291 | } |
291 | 292 | ||
292 | if (replicaInfo.master().get().equals(clusterService.getLocalNode().id())) { | 293 | if (replicaInfo.master().get().equals(clusterService.getLocalNode().id())) { | ... | ... |
-
Please register or login to post a comment