[Proxool-cvs] proxool/src/java/org/logicalcobwebs/proxool/admin Admin.java,1.6,1.7

[<bil...@us...>]

Update of /cvsroot/proxool/proxool/src/java/org/logicalcobwebs/proxool/admin
In directory sc8-pr-cvs1:/tmp/cvs-serv10445/src/java/org/logicalcobwebs/proxool/admin

Modified Files:
	Admin.java 
Log Message:
connectionReturned() and connectionRefused() calls will now log any errors and continue rather than
possibly throwing RuntimeExceptions back to the caller. In principle, stats should not cause problems
to the core code. (No evidence of this happening - but it's more robust now.)

Index: Admin.java
===================================================================
RCS file: /cvsroot/proxool/proxool/src/java/org/logicalcobwebs/proxool/admin/Admin.java,v
retrieving revision 1.6
retrieving revision 1.7
diff -C2 -d -r1.6 -r1.7
*** Admin.java	30 Aug 2003 14:54:04 -0000	1.6
--- Admin.java	27 Oct 2003 20:26:19 -0000	1.7
***************
*** 68,75 ****
       */
      public void connectionReturned(long activeTime) {
!         Iterator i = statsRollers.values().iterator();
!         while (i.hasNext()) {
!             StatsRoller statsRoller = (StatsRoller) i.next();
!             statsRoller.connectionReturned(activeTime);
          }
      }
--- 68,79 ----
       */
      public void connectionReturned(long activeTime) {
!         try {
!             Iterator i = statsRollers.values().iterator();
!             while (i.hasNext()) {
!                 StatsRoller statsRoller = (StatsRoller) i.next();
!                 statsRoller.connectionReturned(activeTime);
!             }
!         } catch (Throwable e) {
!             LOG.error("Stats connectionReturned call failed. Ignoring.", e);
          }
      }
***************
*** 79,86 ****
       */
      public void connectionRefused() {
!         Iterator i = statsRollers.values().iterator();
!         while (i.hasNext()) {
!             StatsRoller statsRoller = (StatsRoller) i.next();
!             statsRoller.connectionRefused();
          }
      }
--- 83,94 ----
       */
      public void connectionRefused() {
!         try {
!             Iterator i = statsRollers.values().iterator();
!             while (i.hasNext()) {
!                 StatsRoller statsRoller = (StatsRoller) i.next();
!                 statsRoller.connectionRefused();
!             }
!         } catch (Exception e) {
!             LOG.error("Stats connectionRefused call failed. Ignoring.", e);
          }
      }
***************
*** 161,164 ****
--- 169,177 ----
   Revision history:
   $Log$
+  Revision 1.7  2003/10/27 20:26:19  billhorsman
+  connectionReturned() and connectionRefused() calls will now log any errors and continue rather than
+  possibly throwing RuntimeExceptions back to the caller. In principle, stats should not cause problems
+  to the core code. (No evidence of this happening - but it's more robust now.)
+ 
   Revision 1.6  2003/08/30 14:54:04  billhorsman
   Checkstyle