Re: svn commit: r1005367 - in /tomcat/trunk: java/org/apache/coyote/ajp/AbstractAjpProtocol.java java/org/apache/coyote/ajp/AjpAprProtocol.java java/org/apache/coyote/ajp/AjpProtocol.java webapps/docs

2010-10-08 Thread Mladen Turk
On 10/07/2010 10:54 AM, kfuj...@apache.org wrote: +/** + * AJP packet size. + */ +protected int packetSize = Constants.MAX_PACKET_SIZE; +public int getPacketSize() { return packetSize; } +public void setPacketSize(int packetSize) { +if(packetSize

Re: svn commit: r1005367 - in /tomcat/trunk: java/org/apache/coyote/ajp/AbstractAjpProtocol.java java/org/apache/coyote/ajp/AjpAprProtocol.java java/org/apache/coyote/ajp/AjpProtocol.java webapps/docs

2010-10-08 Thread Keiichi Fujino
On 10/07/2010 10:54 AM, kfuj...@apache.org wrote: +    /** +     * AJP packet size. +     */ +    protected int packetSize = Constants.MAX_PACKET_SIZE; +    public int getPacketSize() { return packetSize; } +    public void setPacketSize(int packetSize) { +        if(packetSize  

Re: svn commit: r1005367 - in /tomcat/trunk: java/org/apache/coyote/ajp/AbstractAjpProtocol.java java/org/apache/coyote/ajp/AjpAprProtocol.java java/org/apache/coyote/ajp/AjpProtocol.java webapps/docs

2010-10-08 Thread Mladen Turk
On 10/08/2010 09:50 AM, Keiichi Fujino wrote: Does it mean to rename MAX_PACKET_SIZE to MIN_PACKET_SIZE ? Huh, I thought we enlarged the MAX_PACKET_SIZE to 65536. We should have DEF_PACKET_SIZE=8192 (or MIN_PACKET_SIZE) and MAX_PACKET_SIZE=65536 Currently we allow to set packet size to

svn commit: r1005367 - in /tomcat/trunk: java/org/apache/coyote/ajp/AbstractAjpProtocol.java java/org/apache/coyote/ajp/AjpAprProtocol.java java/org/apache/coyote/ajp/AjpProtocol.java webapps/docs/cha

2010-10-07 Thread kfujino
Author: kfujino Date: Thu Oct 7 08:54:51 2010 New Revision: 1005367 URL: http://svn.apache.org/viewvc?rev=1005367view=rev Log: Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=50054 Correctly handle the setting of minSpareThreads in AJP connector. Other attributes (acceptorThreadCount