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