Re: [tomcat] branch master updated: Use async by default for now

2019-04-29 Thread Rémy Maucherat
On Fri, Apr 12, 2019 at 6:15 PM  wrote:

> This is an automated email from the ASF dual-hosted git repository.
>
> remm pushed a commit to branch master
> in repository https://gitbox.apache.org/repos/asf/tomcat.git
>
>
> The following commit(s) were added to refs/heads/master by this push:
>  new 39673ee  Use async by default for now
> 39673ee is described below
>
> commit 39673eeddcfe6930ccca7ada4601720eb74157b2
> Author: remm 
> AuthorDate: Fri Apr 12 18:15:00 2019 +0200
>
> Use async by default for now
>
> Goal: more testing with NIO
>

The testing looks fine so far so I don't plan to switch the default back at
this time.
The new code brings a simplified way to do non blocking frame reads, it's
clearly a good capability to have.

Rémy


> ---
>  java/org/apache/tomcat/util/net/AbstractEndpoint.java | 4 ++--
>  webapps/docs/changelog.xml| 8 
>  webapps/docs/config/http.xml  | 4 ++--
>  3 files changed, 12 insertions(+), 4 deletions(-)
>
> diff --git a/java/org/apache/tomcat/util/net/AbstractEndpoint.java
> b/java/org/apache/tomcat/util/net/AbstractEndpoint.java
> index 155cabe..64be45b 100644
> --- a/java/org/apache/tomcat/util/net/AbstractEndpoint.java
> +++ b/java/org/apache/tomcat/util/net/AbstractEndpoint.java
> @@ -742,9 +742,9 @@ public abstract class AbstractEndpoint {
>
>
>  /**
> - * Expose async IO capability.
> + * Expose asynchronous IO capability.
>   */
> -private boolean useAsyncIO = false;
> +private boolean useAsyncIO = true;
>  public void setUseAsyncIO(boolean useAsyncIO) { this.useAsyncIO =
> useAsyncIO; }
>  public boolean getUseAsyncIO() { return useAsyncIO; }
>
> diff --git a/webapps/docs/changelog.xml b/webapps/docs/changelog.xml
> index 335568d..2542985 100644
> --- a/webapps/docs/changelog.xml
> +++ b/webapps/docs/changelog.xml
> @@ -45,6 +45,14 @@
>issues do not "pop up" wrt. others).
>  -->
>  
> +  
> +
> +  
> +The useAsyncIO boolean attribute on the Connector
> element
> +value now defaults to true. (remm)
> +  
> +
> +  
>  
>  
>
> diff --git a/webapps/docs/config/http.xml b/webapps/docs/config/http.xml
> index fe8ba89..9e68e39 100644
> --- a/webapps/docs/config/http.xml
> +++ b/webapps/docs/config/http.xml
> @@ -762,7 +762,7 @@
>
>
>  (bool)Use this attribute to enable or disable usage of the
> -asynchronous IO API. The default value is false.
> +asynchronous IO API. The default value is true.
>
>
>
> @@ -906,7 +906,7 @@
>
>
>  (bool)Use this attribute to enable or disable usage of the
> -asynchronous IO API. The default value is false.
> +asynchronous IO API. The default value is true.
>
>
>
>
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
> For additional commands, e-mail: dev-h...@tomcat.apache.org
>
>


[tomcat] branch master updated: Use async by default for now

2019-04-12 Thread remm
This is an automated email from the ASF dual-hosted git repository.

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


The following commit(s) were added to refs/heads/master by this push:
 new 39673ee  Use async by default for now
39673ee is described below

commit 39673eeddcfe6930ccca7ada4601720eb74157b2
Author: remm 
AuthorDate: Fri Apr 12 18:15:00 2019 +0200

Use async by default for now

Goal: more testing with NIO
---
 java/org/apache/tomcat/util/net/AbstractEndpoint.java | 4 ++--
 webapps/docs/changelog.xml| 8 
 webapps/docs/config/http.xml  | 4 ++--
 3 files changed, 12 insertions(+), 4 deletions(-)

diff --git a/java/org/apache/tomcat/util/net/AbstractEndpoint.java 
b/java/org/apache/tomcat/util/net/AbstractEndpoint.java
index 155cabe..64be45b 100644
--- a/java/org/apache/tomcat/util/net/AbstractEndpoint.java
+++ b/java/org/apache/tomcat/util/net/AbstractEndpoint.java
@@ -742,9 +742,9 @@ public abstract class AbstractEndpoint {
 
 
 /**
- * Expose async IO capability.
+ * Expose asynchronous IO capability.
  */
-private boolean useAsyncIO = false;
+private boolean useAsyncIO = true;
 public void setUseAsyncIO(boolean useAsyncIO) { this.useAsyncIO = 
useAsyncIO; }
 public boolean getUseAsyncIO() { return useAsyncIO; }
 
diff --git a/webapps/docs/changelog.xml b/webapps/docs/changelog.xml
index 335568d..2542985 100644
--- a/webapps/docs/changelog.xml
+++ b/webapps/docs/changelog.xml
@@ -45,6 +45,14 @@
   issues do not "pop up" wrt. others).
 -->
 
+  
+
+  
+The useAsyncIO boolean attribute on the Connector element
+value now defaults to true. (remm)
+  
+
+  
 
 
   
diff --git a/webapps/docs/config/http.xml b/webapps/docs/config/http.xml
index fe8ba89..9e68e39 100644
--- a/webapps/docs/config/http.xml
+++ b/webapps/docs/config/http.xml
@@ -762,7 +762,7 @@
 
   
 (bool)Use this attribute to enable or disable usage of the
-asynchronous IO API. The default value is false.
+asynchronous IO API. The default value is true.
   
 
   
@@ -906,7 +906,7 @@
 
   
 (bool)Use this attribute to enable or disable usage of the
-asynchronous IO API. The default value is false.
+asynchronous IO API. The default value is true.
   
 
   


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