Add proper break statements to abstract reader

Change-Id: Iebb52d59e2e342151e616548722366985d7ee9a5
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

1 files changed, 3 insertions, 2 deletions

diff --git a/v3po/v3po2vpp/src/main/java/io/fd/honeycomb/v3po/translate/v3po/interfacesstate/InterfaceUtils.java b/v3po/v3po2vpp/src/main/java/io/fd/honeycomb/v3po/translate/v3po/interfacesstate/InterfaceUtils.java
index 09ea09e47..3db497a5c 100644
--- a/v3po/v3po2vpp/src/main/java/io/fd/honeycomb/v3po/translate/v3po/interfacesstate/InterfaceUtils.java
+++ b/v3po/v3po2vpp/src/main/java/io/fd/honeycomb/v3po/translate/v3po/interfacesstate/InterfaceUtils.java
@@ -162,8 +162,6 @@ public final class InterfaceUtils {
         CompletionStage<SwInterfaceDetailsReplyDump> requestFuture = futureJvpp.swInterfaceDump(request);
         SwInterfaceDetailsReplyDump ifaces = V3poUtils.getReply(requestFuture.toCompletableFuture());
         if (null == ifaces || null == ifaces.swInterfaceDetails || ifaces.swInterfaceDetails.isEmpty()) {
-            LOG.warn("VPP returned null instead of interface by key {}", key.getName());
-            LOG.warn("Iterating through all the interfaces to find interface: {}", key.getName());
             request.nameFilterValid = 0;
 
             // Returned cached if available
@@ -171,6 +169,9 @@ public final class InterfaceUtils {
                 return allInterfaces.get(index);
             }
 
+            LOG.warn("VPP returned null instead of interface by key {} and its not cached", key.getName());
+            LOG.warn("Iterating through all the interfaces to find interface: {}", key.getName());
+
             // Or else just perform full dump and do inefficient filtering
             requestFuture = futureJvpp.swInterfaceDump(request);
             ifaces = V3poUtils.getReply(requestFuture.toCompletableFuture());