This is an automated email from the ASF dual-hosted git repository.

remm pushed a commit to branch 9.0.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git


The following commit(s) were added to refs/heads/9.0.x by this push:
     new d2c6c6e875 Change default value for discardFacades to true
d2c6c6e875 is described below

commit d2c6c6e875cf049f7d7aca3506c2e1e337511a63
Author: remm <r...@apache.org>
AuthorDate: Mon Jun 10 17:06:00 2024 +0200

    Change default value for discardFacades to true
---
 java/org/apache/catalina/connector/Connector.java | 2 +-
 webapps/docs/changelog.xml                        | 8 ++++++++
 webapps/docs/config/http.xml                      | 2 +-
 webapps/docs/config/systemprops.xml               | 2 +-
 4 files changed, 11 insertions(+), 3 deletions(-)

diff --git a/java/org/apache/catalina/connector/Connector.java 
b/java/org/apache/catalina/connector/Connector.java
index ee1e94edd7..0e7b3ecd55 100644
--- a/java/org/apache/catalina/connector/Connector.java
+++ b/java/org/apache/catalina/connector/Connector.java
@@ -65,7 +65,7 @@ public class Connector extends LifecycleMBeanBase {
      * Alternate flag to enable recycling of facades.
      */
     public static final boolean RECYCLE_FACADES =
-            
Boolean.parseBoolean(System.getProperty("org.apache.catalina.connector.RECYCLE_FACADES",
 "false"));
+            
Boolean.parseBoolean(System.getProperty("org.apache.catalina.connector.RECYCLE_FACADES",
 "true"));
 
 
     public static final String INTERNAL_EXECUTOR_NAME = "Internal";
diff --git a/webapps/docs/changelog.xml b/webapps/docs/changelog.xml
index 0e06c11a6c..27569bd081 100644
--- a/webapps/docs/changelog.xml
+++ b/webapps/docs/changelog.xml
@@ -122,6 +122,14 @@
         Update minimum recommended version of Tomcat Native to 1.3.0. Pull
         request <pr>728</pr> provided by Dimitrios Soumis. (markt)
       </update>
+      <update>
+        The system property
+        <code>org.apache.catalina.connector.RECYCLE_FACADES</code> will now
+        default to <code>true</code> if not specified, which will in turn set
+        the default value for the <code>discardFacades</code> connector
+        attribute, thus causing facade objects to be discarded by default.
+        (remm)
+      </update>
     </changelog>
   </subsection>
   <subsection name="Coyote">
diff --git a/webapps/docs/config/http.xml b/webapps/docs/config/http.xml
index 1cfde6b55e..dcd1778d53 100644
--- a/webapps/docs/config/http.xml
+++ b/webapps/docs/config/http.xml
@@ -100,7 +100,7 @@
       reused. This setting has no effect when the security manager is enabled.
       If not specified, this attribute is set to the value of the
       <code>org.apache.catalina.connector.RECYCLE_FACADES</code> system
-      property, or <code>false</code> if not set.</p>
+      property, or <code>true</code> if not set.</p>
     </attribute>
 
     <attribute name="enableLookups" required="false">
diff --git a/webapps/docs/config/systemprops.xml 
b/webapps/docs/config/systemprops.xml
index 6f53cb9afa..d80518e7e9 100644
--- a/webapps/docs/config/systemprops.xml
+++ b/webapps/docs/config/systemprops.xml
@@ -284,7 +284,7 @@
     <property name="org.apache.catalina.connector. RECYCLE_FACADES">
       <p>If this is <code>true</code> or if a security manager is in use a new
       facade object will be created for each request.</p>
-      <p>If not specified, the default value of <code>false</code> will be 
used.</p>
+      <p>If not specified, the default value of <code>true</code> will be 
used.</p>
     </property>
 
     <property


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to