Author: rjung
Date: Fri Oct  3 09:35:57 2008
New Revision: 701435

URL: http://svn.apache.org/viewvc?rev=701435&view=rev
Log:
Minor watchdig enhancements:
- Reset watchdog interval to 0, if thread creation
  fails (so we use the request based maintenance)
  and any partially initialized thread might stop.
- Let Apache log similar messages as IIS
- Check and correct negative interval values in IIS like
  we do in Apache
- Use JK log for thread creation error message, because
  at that time we already have it available.

Modified:
    tomcat/connectors/trunk/jk/native/apache-2.0/mod_jk.c
    tomcat/connectors/trunk/jk/native/iis/jk_isapi_plugin.c

Modified: tomcat/connectors/trunk/jk/native/apache-2.0/mod_jk.c
URL: 
http://svn.apache.org/viewvc/tomcat/connectors/trunk/jk/native/apache-2.0/mod_jk.c?rev=701435&r1=701434&r2=701435&view=diff
==============================================================================
--- tomcat/connectors/trunk/jk/native/apache-2.0/mod_jk.c (original)
+++ tomcat/connectors/trunk/jk/native/apache-2.0/mod_jk.c Fri Oct  3 09:35:57 
2008
@@ -2820,7 +2820,8 @@
 
     if (JK_IS_DEBUG_LEVEL(conf->log))
         jk_log(conf->log, JK_LOG_DEBUG,
-               "Watchdog initialized");
+               "Watchdog thread initialized with %d second interval",
+               jk_watchdog_interval);
     for (;;) {
         for (i = 0; i < jk_watchdog_interval; i++) {
             apr_sleep(apr_time_from_sec(1));
@@ -2831,7 +2832,7 @@
             break;
         if (JK_IS_DEBUG_LEVEL(conf->log))
            jk_log(conf->log, JK_LOG_DEBUG,
-                  "Watchdog running");
+                  "Watchdog thread running");
         wc_maintain(conf->log);
     }
     return NULL;
@@ -2862,8 +2863,9 @@
 #if APR_HAS_THREADS
         rv = apr_thread_create(&wdt, NULL, jk_watchdog_func, conf, pconf);
         if (rv != APR_SUCCESS) {
-            ap_log_error(APLOG_MARK, APLOG_CRIT, rv, s,
-                     "mod_jk: could not init JK watchdog thread");
+            jk_log(conf->log, JK_LOG_ERROR,
+                   "Could not init watchdog thread, error=%d", rv);
+            jk_watchdog_interval = 0;
         }
         apr_thread_detach(wdt);
 #endif

Modified: tomcat/connectors/trunk/jk/native/iis/jk_isapi_plugin.c
URL: 
http://svn.apache.org/viewvc/tomcat/connectors/trunk/jk/native/iis/jk_isapi_plugin.c?rev=701435&r1=701434&r2=701435&view=diff
==============================================================================
--- tomcat/connectors/trunk/jk/native/iis/jk_isapi_plugin.c (original)
+++ tomcat/connectors/trunk/jk/native/iis/jk_isapi_plugin.c Fri Oct  3 09:35:57 
2008
@@ -2579,6 +2579,8 @@
 #endif
     reject_unsafe = get_config_bool(src, REJECT_UNSAFE_TAG, JK_FALSE);
     watchdog_interval = get_config_int(src, WATCHDOG_INTERVAL_TAG, 0);
+    if (watchdog_interval < 0)
+        watchdog_interval = 0;
 #ifdef ISAPI_ALLOW_CHUNKING
     chunked_encoding_enabled = get_config_bool(src, 
ENABLE_CHUNKED_ENCODING_TAG, JK_FALSE);
 #endif



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to