cxf-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Willem Jiang <willem.ji...@gmail.com>
Subject Re: JettyHTTPServerEngine changes (r1177165 - in /cxf/branches/2.4.x-fixes...)
Date Fri, 30 Sep 2011 01:30:40 GMT
On Fri Sep 30 05:39:54 2011, Glen Mazza wrote:
> Hi Willem:
>
> On 09/29/2011 01:05 AM, ningjiang@apache.org wrote:
>> Author: ningjiang
>> Date: Thu Sep 29 05:05:18 2011
>> New Revision: 1177165
>>
>> Propchange: cxf/branches/2.4.x-fixes/
>> ------------------------------------------------------------------------------ 
>>
>> Binary property 'svnmerge-integrated' - no diff available.
>>
>> Modified: 
>> cxf/branches/2.4.x-fixes/rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/JettyHTTPServerEngine.java

>>
>> URL: 
>> http://svn.apache.org/viewvc/cxf/branches/2.4.x-fixes/rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/JettyHTTPServerEngine.java?rev=1177165&r1=1177164&r2=1177165&view=diff

>>
>> ============================================================================== 
>>
>> --- 
>> cxf/branches/2.4.x-fixes/rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/JettyHTTPServerEngine.java

>> (original)
>> +++ 
>> cxf/branches/2.4.x-fixes/rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/JettyHTTPServerEngine.java

>> Thu Sep 29 05:05:18 2011
>> @@ -95,6 +95,7 @@ public class JettyHTTPServerEngine
>> private Boolean isSessionSupport = false;
>> private Boolean isReuseAddress = true;
>> private Boolean continuationsEnabled = true;
>> + private int maxIdleTime = 200000;
>> private int servantCount;
>> private Server server;
>> private Connector connector;
>> @@ -281,6 +282,14 @@ public class JettyHTTPServerEngine
>> isReuseAddress = reuse;
>> }
>> +
>> + public void setMaxIdleTime(int maxIdel) {
>> + maxIdleTime = maxIdel;
>> + }
>
> (int maxIdle)
>
>>
>> /**
>>
>> Modified: 
>> cxf/branches/2.4.x-fixes/rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/spring/JettyHTTPServerEngineBeanDefinitionParser.java

>>
>> URL: 
>> http://svn.apache.org/viewvc/cxf/branches/2.4.x-fixes/rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/spring/JettyHTTPServerEngineBeanDefinitionParser.java?rev=1177165&r1=1177164&r2=1177165&view=diff

>>
>> ============================================================================== 
>>
>> --- 
>> cxf/branches/2.4.x-fixes/rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/spring/JettyHTTPServerEngineBeanDefinitionParser.java

>> (original)
>> +++ 
>> cxf/branches/2.4.x-fixes/rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/spring/JettyHTTPServerEngineBeanDefinitionParser.java

>> Thu Sep 29 05:05:18 2011
>> @@ -70,6 +70,11 @@ public class JettyHTTPServerEngineBeanDe
>> if (continuationsStr != null&& continuationsStr.length()> 0) {
>> bean.addPropertyValue("continuationsEnabled", continuationsStr);
>> }
>> +
>> + String maxIdleTimeStr = element.getAttribute("maxIdleTime");
>> + if (maxIdleTimeStr != null&& !"".equals(maxIdleTimeStr.trim())) {
>> + bean.addPropertyValue("maxIdleTime", maxIdleTimeStr);
>> + }
>> ValueHolder busValue = ctx.getContainingBeanDefinition()
>> .getConstructorArgumentValues().getArgumentValue(0, Bus.class);
>> bean.addPropertyValue("bus", busValue.getValue());
>>
>>
>> Modified: 
>> cxf/branches/2.4.x-fixes/rt/transports/http-jetty/src/main/resources/schemas/configuration/http-jetty.xsd

>>
>> URL: 
>> http://svn.apache.org/viewvc/cxf/branches/2.4.x-fixes/rt/transports/http-jetty/src/main/resources/schemas/configuration/http-jetty.xsd?rev=1177165&r1=1177164&r2=1177165&view=diff

>>
>> ============================================================================== 
>>
>> --- 
>> cxf/branches/2.4.x-fixes/rt/transports/http-jetty/src/main/resources/schemas/configuration/http-jetty.xsd

>> (original)
>> +++ 
>> cxf/branches/2.4.x-fixes/rt/transports/http-jetty/src/main/resources/schemas/configuration/http-jetty.xsd

>> Thu Sep 29 05:05:18 2011
>> @@ -144,9 +144,14 @@
>> <xs:attribute name="continuationsEnabled" 
>> type="ptp:ParameterizedBoolean">
>> <xs:annotation>
>> <xs:documentation>Specifies if Jetty Continuations will be explicitly 
>> supported
>> - by Jetty destinations. Continuations will be checked if this 
>> attribute is set to true or omitted, ignored 
>> otherwise</xs:documentation>
>> + by Jetty destinations. Continuations will be checked if this 
>> attribute is set to true or omitted, ignored 
>> otherwise.</xs:documentation>
>> </xs:annotation>
>> </xs:attribute>
>> +<xs:attribute name="maxIdleTime" type="ptp:ParameterizedInt">
>> +<xs:annotation>
>> +<xs:documentation>Set the maximum Idle time for a connection, the 
>> timer will be reset if there is any read and write option on the 
>> under layer stream.</xs:documentation>
>
> I'm not sure what you mean here by "option" and "under layer 
> (underlying?) stream"--although it could be my lack of knowledge on 
> the topic.
>
After read the sentence twice, I think the option should be action, 
which means if there is no any data is received or sent, the stream 
will be closed.

> Regards,
> Glen
>
>
>
>



-- 
Willem
----------------------------------
FuseSource
Web: http://www.fusesource.com
Blog:    http://willemjiang.blogspot.com (English)
         http://jnn.javaeye.com (Chinese)
Twitter: willemjiang 
Weibo: willemjiang 


Mime
View raw message