Re: svn commit: r1165338 - in /tomcat/trunk/java/org/apache/coyote/http11: AbstractHttp11Processor.java Http11AprProcessor.java Http11NioProcessor.java Http11Processor.java

2011-09-05 Thread Felix Schumacher
ma...@apache.org schrieb: >Author: markt >Date: Mon Sep 5 15:31:46 2011 >New Revision: 1165338 > >URL: http://svn.apache.org/viewvc?rev=1165338&view=rev >Log: >Align request processing between the connectors > >Modified: >tomcat/trunk/java/org/apache/coyote/http11/AbstractHttp11Processor.java >

svn commit: r1165368 - in /tomcat/trunk/java/org/apache/coyote/http11: Http11AprProcessor.java Http11NioProcessor.java Http11Processor.java

2011-09-05 Thread markt
Author: markt Date: Mon Sep 5 17:25:31 2011 New Revision: 1165368 URL: http://svn.apache.org/viewvc?rev=1165368&view=rev Log: Align while loop test Modified: tomcat/trunk/java/org/apache/coyote/http11/Http11AprProcessor.java tomcat/trunk/java/org/apache/coyote/http11/Http11NioProcessor.j

svn commit: r1165367 - in /tomcat/trunk/java/org/apache/coyote/http11: Http11AprProcessor.java Http11Processor.java

2011-09-05 Thread markt
Author: markt Date: Mon Sep 5 17:18:50 2011 New Revision: 1165367 URL: http://svn.apache.org/viewvc?rev=1165367&view=rev Log: Headers should be read with the standard connection timeout. The upload timeout should only be used for the request body. Modified: tomcat/trunk/java/org/apache/coyot

svn commit: r1165361 - in /tomcat/trunk/java/org/apache/coyote/http11: Http11AprProcessor.java Http11Processor.java

2011-09-05 Thread markt
Author: markt Date: Mon Sep 5 16:59:27 2011 New Revision: 1165361 URL: http://svn.apache.org/viewvc?rev=1165361&view=rev Log: Add debug log message to align HTTP connectors Modified: tomcat/trunk/java/org/apache/coyote/http11/Http11AprProcessor.java tomcat/trunk/java/org/apache/coyote/ht

svn commit: r1165360 - /tomcat/trunk/java/org/apache/coyote/http11/Http11NioProcessor.java

2011-09-05 Thread markt
Author: markt Date: Mon Sep 5 16:54:25 2011 New Revision: 1165360 URL: http://svn.apache.org/viewvc?rev=1165360&view=rev Log: Start timing request processing after the request line has been read and before the request headers are read (aligns with other HTTP connectors) Modified: tomcat/tru

svn commit: r1165353 - /tomcat/trunk/webapps/docs/config/http.xml

2011-09-05 Thread markt
Author: markt Date: Mon Sep 5 16:35:17 2011 New Revision: 1165353 URL: http://svn.apache.org/viewvc?rev=1165353&view=rev Log: Revert r1165237. Documented defaults are correct. Modified: tomcat/trunk/webapps/docs/config/http.xml Modified: tomcat/trunk/webapps/docs/config/http.xml URL: http:

svn commit: r1165350 - in /tomcat/tc7.0.x/trunk: ./ webapps/docs/changelog.xml webapps/docs/config/http.xml

2011-09-05 Thread markt
Author: markt Date: Mon Sep 5 16:31:39 2011 New Revision: 1165350 URL: http://svn.apache.org/viewvc?rev=1165350&view=rev Log: Revert r1165240. Documented defaults are correct. Modified: tomcat/tc7.0.x/trunk/ (props changed) tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml tomcat/tc7

svn commit: r1165347 - in /tomcat/trunk/java/org/apache/coyote/http11: AbstractHttp11Processor.java Http11AprProcessor.java Http11Processor.java

2011-09-05 Thread markt
Author: markt Date: Mon Sep 5 16:14:46 2011 New Revision: 1165347 URL: http://svn.apache.org/viewvc?rev=1165347&view=rev Log: Tab police (rebuilt laptop - need to fix Eclipse config) Modified: tomcat/trunk/java/org/apache/coyote/http11/AbstractHttp11Processor.java tomcat/trunk/java/org/a

svn commit: r1165338 - in /tomcat/trunk/java/org/apache/coyote/http11: AbstractHttp11Processor.java Http11AprProcessor.java Http11NioProcessor.java Http11Processor.java

2011-09-05 Thread markt
Author: markt Date: Mon Sep 5 15:31:46 2011 New Revision: 1165338 URL: http://svn.apache.org/viewvc?rev=1165338&view=rev Log: Align request processing between the connectors Modified: tomcat/trunk/java/org/apache/coyote/http11/AbstractHttp11Processor.java tomcat/trunk/java/org/apache/coy

Re: svn commit: r1165240 - in /tomcat/tc7.0.x/trunk: ./ webapps/docs/changelog.xml webapps/docs/config/http.xml

2011-09-05 Thread Mark Thomas
On 05/09/2011 15:59, Konstantin Kolinko wrote: > 2011/9/5 Mark Thomas : >> On 05/09/2011 15:27, Konstantin Kolinko wrote: >>> Why are you changing docs without changing the code? >>> >>> As far as I see >>> org.apache.coyote.http11.Constants.DEFAULT_CONNECTION_TIMEOUT >>> is 6 in 7.0.x. >> >> S

svn commit: r1165331 - in /tomcat/trunk/java/org/apache/coyote/http11: AbstractHttp11Processor.java Http11AprProcessor.java Http11NioProcessor.java Http11Processor.java

2011-09-05 Thread markt
Author: markt Date: Mon Sep 5 15:21:22 2011 New Revision: 1165331 URL: http://svn.apache.org/viewvc?rev=1165331&view=rev Log: Complete alignment of process() for finishing requests Modified: tomcat/trunk/java/org/apache/coyote/http11/AbstractHttp11Processor.java tomcat/trunk/java/org/apa

svn commit: r1165309 - in /tomcat/trunk/java/org/apache/coyote/http11: AbstractHttp11Processor.java Http11AprProcessor.java Http11NioProcessor.java Http11Processor.java

2011-09-05 Thread markt
Author: markt Date: Mon Sep 5 15:03:51 2011 New Revision: 1165309 URL: http://svn.apache.org/viewvc?rev=1165309&view=rev Log: Pull out implementation specific processing that determines if keep-alive should be broken into a separate method. Modified: tomcat/trunk/java/org/apache/coyote/http

Re: svn commit: r1165240 - in /tomcat/tc7.0.x/trunk: ./ webapps/docs/changelog.xml webapps/docs/config/http.xml

2011-09-05 Thread Konstantin Kolinko
2011/9/5 Mark Thomas : > On 05/09/2011 15:27, Konstantin Kolinko wrote: >> Why are you changing docs without changing the code? >> >> As far as I see >> org.apache.coyote.http11.Constants.DEFAULT_CONNECTION_TIMEOUT >> is 6 in 7.0.x. > > Sorry, sent that last reply before I read your message pro

Re: svn commit: r1165240 - in /tomcat/tc7.0.x/trunk: ./ webapps/docs/changelog.xml webapps/docs/config/http.xml

2011-09-05 Thread Mark Thomas
On 05/09/2011 15:27, Konstantin Kolinko wrote: > Why are you changing docs without changing the code? > > As far as I see > org.apache.coyote.http11.Constants.DEFAULT_CONNECTION_TIMEOUT > is 6 in 7.0.x. Sorry, sent that last reply before I read your message properly. I was going of the defaul

Re: svn commit: r1165240 - in /tomcat/tc7.0.x/trunk: ./ webapps/docs/changelog.xml webapps/docs/config/http.xml

2011-09-05 Thread Mark Thomas
On 05/09/2011 15:27, Konstantin Kolinko wrote: > Why are you changing docs without changing the code? Because the code is 2 unless I have missed something. Mark > > As far as I see > org.apache.coyote.http11.Constants.DEFAULT_CONNECTION_TIMEOUT > is 6 in 7.0.x. > > Best regards, > Kons

Re: svn commit: r1165240 - in /tomcat/tc7.0.x/trunk: ./ webapps/docs/changelog.xml webapps/docs/config/http.xml

2011-09-05 Thread Konstantin Kolinko
Why are you changing docs without changing the code? As far as I see org.apache.coyote.http11.Constants.DEFAULT_CONNECTION_TIMEOUT is 6 in 7.0.x. Best regards, Konstantin Kolinko 2011/9/5 : > Author: markt > Date: Mon Sep  5 11:06:06 2011 > New Revision: 1165240 > > URL: http://svn.apache.o

[Tomcat Wiki] Update of "LocalBadContent" by ChuckCaldarale

2011-09-05 Thread Apache Wiki
Dear Wiki user, You have subscribed to a wiki page or wiki category on "Tomcat Wiki" for change notification. The "LocalBadContent" page has been changed by ChuckCaldarale: http://wiki.apache.org/tomcat/LocalBadContent?action=diff&rev1=28&rev2=29 isas\.com\.cn j\-cellular kogaryu\.com + l

[Tomcat Wiki] Update of "LocalBadContent" by ChuckCaldarale

2011-09-05 Thread Apache Wiki
Dear Wiki user, You have subscribed to a wiki page or wiki category on "Tomcat Wiki" for change notification. The "LocalBadContent" page has been changed by ChuckCaldarale: http://wiki.apache.org/tomcat/LocalBadContent?action=diff&rev1=27&rev2=28 easyday\.cn electronic-wire\.com envy\.nu

svn commit: r1165282 - in /tomcat/trunk/java/org/apache/coyote/http11: AbstractHttp11Processor.java Http11AprProcessor.java Http11NioProcessor.java Http11Processor.java

2011-09-05 Thread markt
Author: markt Date: Mon Sep 5 13:45:14 2011 New Revision: 1165282 URL: http://svn.apache.org/viewvc?rev=1165282&view=rev Log: Refactoring prep for extracting sendfile code to a separate method Modified: tomcat/trunk/java/org/apache/coyote/http11/AbstractHttp11Processor.java tomcat/trunk/

svn commit: r1165273 - in /tomcat/trunk/java/org/apache/coyote/http11: Http11AprProcessor.java Http11NioProcessor.java Http11Processor.java

2011-09-05 Thread markt
Author: markt Date: Mon Sep 5 13:21:28 2011 New Revision: 1165273 URL: http://svn.apache.org/viewvc?rev=1165273&view=rev Log: Move stage update to before point where we might break out of the keep-alive loop Modified: tomcat/trunk/java/org/apache/coyote/http11/Http11AprProcessor.java to

svn commit: r1165253 - in /tomcat/trunk/java/org/apache/coyote/http11: Http11AprProcessor.java Http11NioProcessor.java Http11Processor.java

2011-09-05 Thread markt
Author: markt Date: Mon Sep 5 12:09:28 2011 New Revision: 1165253 URL: http://svn.apache.org/viewvc?rev=1165253&view=rev Log: Start to align the request finish code Modified: tomcat/trunk/java/org/apache/coyote/http11/Http11AprProcessor.java tomcat/trunk/java/org/apache/coyote/http11/Htt

svn commit: r1165248 - /tomcat/trunk/java/org/apache/coyote/http11/Http11Processor.java

2011-09-05 Thread markt
Author: markt Date: Mon Sep 5 11:57:05 2011 New Revision: 1165248 URL: http://svn.apache.org/viewvc?rev=1165248&view=rev Log: Remove unnecessary try/catch that just wraps a simple setter Modified: tomcat/trunk/java/org/apache/coyote/http11/Http11Processor.java Modified: tomcat/trunk/java/or

svn commit: r1165247 - /tomcat/trunk/java/org/apache/coyote/http11/Http11Processor.java

2011-09-05 Thread markt
Author: markt Date: Mon Sep 5 11:55:31 2011 New Revision: 1165247 URL: http://svn.apache.org/viewvc?rev=1165247&view=rev Log: Remove unnecessary try/catch The methods it contained were either simple setters or already provided exactly the same exception handling Modified: tomcat/trunk/java/

svn commit: r1165240 - in /tomcat/tc7.0.x/trunk: ./ webapps/docs/changelog.xml webapps/docs/config/http.xml

2011-09-05 Thread markt
Author: markt Date: Mon Sep 5 11:06:06 2011 New Revision: 1165240 URL: http://svn.apache.org/viewvc?rev=1165240&view=rev Log: Correct documented default for connectionTimeout with HTTP connectors Modified: tomcat/tc7.0.x/trunk/ (props changed) tomcat/tc7.0.x/trunk/webapps/docs/changelo

svn commit: r1165237 - /tomcat/trunk/webapps/docs/config/http.xml

2011-09-05 Thread markt
Author: markt Date: Mon Sep 5 10:57:13 2011 New Revision: 1165237 URL: http://svn.apache.org/viewvc?rev=1165237&view=rev Log: Correct documentation of default connection timeout Modified: tomcat/trunk/webapps/docs/config/http.xml Modified: tomcat/trunk/webapps/docs/config/http.xml URL: htt

svn commit: r1165236 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/coyote/http11/Http11Processor.java webapps/docs/changelog.xml

2011-09-05 Thread markt
Author: markt Date: Mon Sep 5 10:56:04 2011 New Revision: 1165236 URL: http://svn.apache.org/viewvc?rev=1165236&view=rev Log: Further re-factoring of the HTTP connectors to align the BIO, NIO and APR implementations. Modified: tomcat/tc7.0.x/trunk/ (props changed) tomcat/tc7.0.x/trun

svn commit: r1165235 - /tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml

2011-09-05 Thread markt
Author: markt Date: Mon Sep 5 10:53:48 2011 New Revision: 1165235 URL: http://svn.apache.org/viewvc?rev=1165235&view=rev Log: line length Modified: tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml Modified: tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml URL: http://svn.apache.org/viewvc/to

svn commit: r1165234 - /tomcat/trunk/java/org/apache/coyote/http11/Http11Processor.java

2011-09-05 Thread markt
Author: markt Date: Mon Sep 5 10:47:43 2011 New Revision: 1165234 URL: http://svn.apache.org/viewvc?rev=1165234&view=rev Log: Remove unnecessary code Timeout is set when socket is created and further changes are handled inside the while loop Modified: tomcat/trunk/java/org/apache/coyote/htt

DO NOT REPLY [Bug 51756] Question on driver URL Tomcat JDBC Pool with Sybase

2011-09-05 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=51756 Mark Thomas changed: What|Removed |Added Status|NEW |RESOLVED Resolution|

DO NOT REPLY [Bug 51756] Question on driver URL Tomcat JDBC Pool with Sybase

2011-09-05 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=51756 windson.rangavajh...@gmail.com changed: What|Removed |Added Summary|Tomcat JDBC Pool does not |Question on driver

DO NOT REPLY [Bug 47203] Make JMXAdaptorLifecycleListener Java 1.4 compatible

2011-09-05 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=47203 Mark Thomas changed: What|Removed |Added Status|NEW |RESOLVED Resolution|

DO NOT REPLY [Bug 45052] Provide read only access for certain role in Manager webapp

2011-09-05 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=45052 Mark Thomas changed: What|Removed |Added Status|NEW |RESOLVED Resolution|

DO NOT REPLY [Bug 48392] jdbc-pool is not returning the proxied connection in resultSet and statement

2011-09-05 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=48392 Neil Clarke changed: What|Removed |Added Status|RESOLVED|REOPENED Resolution|FIXED

DO NOT REPLY [Bug 48392] jdbc-pool is not returning the proxied connection in resultSet and statement

2011-09-05 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=48392 Neil Clarke changed: What|Removed |Added Attachment #25054|0 |1 is obsolete|