Findbugs fixes

Testing Sonar with a few findbugs fixes.  Suppressing a false
positive to see if it works.

Change-Id: I0cdf2921198787adc56c680840aa00bdd254269b

@@ -210,7 +210,12 @@ public class FooComponent {
                 }
                 }
             }
             }
             int retry = 5;
             int retry = 5;
+
             do {
             do {
+                if (vv == null) {
+                    log.error("Shouldn't reach here - value is null");
+                    break;
+                }
                 ByteBuffer prev = ByteBuffer.wrap(vv.value());
                 ByteBuffer prev = ByteBuffer.wrap(vv.value());
                 long next = prev.getLong() + 1;
                 long next = prev.getLong() + 1;
                 byte[] newValue = ByteBuffer.allocate(Long.BYTES).putLong(next).array();
                 byte[] newValue = ByteBuffer.allocate(Long.BYTES).putLong(next).array();

@@ -94,6 +94,9 @@ public class ClusterMessagingProtocolClient implements ProtocolClient {
     }
     }
 
 
     @Override
     @Override
+//    @edu.umd.cs.findbugs.annotations.SuppressWarnings
+//            (value="NP_NONNULL_PARAM_VIOLATION",
+//             justification="False positives on completedFuture call, passing null as a parameter is allowed")
     public synchronized CompletableFuture<Void> connect() {
     public synchronized CompletableFuture<Void> connect() {
         if (remoteNode != null) {
         if (remoteNode != null) {
             // done
             // done