- Collector #1661: wire up 'python-check-interval'.

doc/CHANGES.txt

@@ -51,6 +51,10 @@ Zope Changes
 
     Bugs fixed
 
+      - Collector #1661: make 'python-check-interval' setting in zope.conf
+        actually work as documented.  This setting allows for important
+        tuning opportunities for production Zope servers.
+
       - Collector #1657:  Don't break host-based virtual hosting when
         purging an HTTP accelerator.
 

lib/python/Zope/Startup/__init__.py

@@ -94,6 +94,7 @@ class ZopeStarter:
         self.dropPrivileges()
         self.makeLockFile()
         self.makePidFile()
+        self.setupInterpreter()
         self.startZope()
         self.registerSignals()
         # emit a "ready" message in order to prevent the kinds of emails
@@ -306,6 +307,10 @@ class ZopeStarter:
             except OSError:
                 pass
 
+    def setupInterpreter(self):
+        """ make changes to the python interpreter environment """
+        sys.setcheckinterval(self.cfg.python_check_interval)
+
 
 class WindowsZopeStarter(ZopeStarter):
 

lib/python/Zope/Startup/tests/testStarter.py

@@ -356,6 +356,22 @@ class ZopeStarterTestCase(test_logger.LoggingTestBase):
             starter.unlinkPidFile()
             self.failIf(os.path.exists(name))
 
+    def testConfigureInterpreter(self):
+        import sys
+        oldcheckinterval = sys.getcheckinterval()
+        newcheckinterval = oldcheckinterval + 1
+        conf = self.load_config_text("""
+                    instancehome <<INSTANCE_HOME>>
+                    python-check-interval %d
+                    """ %  newcheckinterval  
+        )
+        try:
+            starter = self.get_starter(conf)
+            starter.setupInterpreter()
+            self.failUnlessEqual( sys.getcheckinterval() , newcheckinterval )
+        finally:
+            sys.setcheckinterval(oldcheckinterval)
+
     def testZopeRunConfigure(self):
         old_config = getConfiguration()
         try: