bug 14301: suppress expected exceptions from showing annoying stack dump, the message will be logged at TRACE level.

Kelven Yang · Kelven Yang · commit 16ab819502ff · 2012-03-28T15:47:00.000-07:00
Reviewed-By: Kelven
diff --git a/server/src/com/cloud/cluster/ClusterServiceServletContainer.java b/server/src/com/cloud/cluster/ClusterServiceServletContainer.java
@@ -137,10 +137,12 @@ public void run() {
 	                            	if(s_logger.isTraceEnabled())
 	                            		s_logger.trace("Cluster request from " + conn.getRemoteAddress().toString() + " is processed");
                             	}
-                            } catch (ConnectionClosedException ex) {
-                                s_logger.error("Client closed connection", ex);
+                            } catch (ConnectionClosedException ex) {
+                            	// client close and read time out exceptions are expected
+                            	// when KEEP-AVLIE is enabled
+                                s_logger.trace("Client closed connection", ex);
                             } catch (IOException ex) {
-                                s_logger.error("I/O error", ex);
+                                s_logger.trace("I/O error", ex);
                             } catch (HttpException ex) {
                                 s_logger.error("Unrecoverable HTTP protocol violation", ex);
                             } finally {
