Minor tweak to reduce INFO logging for KeyedResourcePool::attemptGrow

Omega1 · Jan 15, 2013 · 7b8d30c · 7b8d30c
1 parent 515b139
commit 7b8d30c
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/src/java/voldemort/utils/pool/KeyedResourcePool.java b/src/java/voldemort/utils/pool/KeyedResourcePool.java
@@ -448,17 +448,21 @@ public <K> boolean attemptGrow(K key, ResourceFactory<K, V> objectFactory) throw
                         if(!nonBlockingPut(resource)) {
                             this.size.decrementAndGet();
                             objectFactory.destroy(key, resource);
+                            if (currentCreatesInFlight>0) {
                             logger.info("attemptGrow established new connection for key "
                                         + key.toString() + " with " + currentCreatesInFlight
                                         + " other connection establishments in flight."
                                         + " And then promptly destroyed the new connection.");
+                            }
                             return false;
                         }
+                        if (currentCreatesInFlight>0) {
                         logger.info("attemptGrow established new connection for key "
                                     + key.toString() + " with " + currentCreatesInFlight
                                     + " other connection establishments in flight."
                                     + " After checking in to KeyedResourcePool, there are "
                                     + queue.size() + " destinations checked in.");
+                        }
                     }
                 } catch(Exception e) {
                     // If nonBlockingPut throws an exception, then we could leak