On 08/06/18 15:40, Rémy Maucherat wrote:
> On Fri, Jun 8, 2018 at 4:10 PM <ma...@apache.org> wrote:
> 
>> +1.  Remove the use of system properties to control configuration wherever
>> +    possible.
>>
> 
> Ok. My bad, I'm a recovering addict ... ;)
Grin.

>> +2.  Reduce instances of setters and getters for the same property
>> existing on an
>> +    object and its parent. This may require new objects to be exposed via
>> JMX.
>>
> 
> Ok. I'm not sure where though, just like that.

I think a lot of this will go away when the deprecated Connector code is
removed. Beyond that there are bits an pieces of duplication. I think of
this entry more as something to look at if there is some spare time at a
point where we can still change the API.

>> +3.  Consider wrapping the SocketWrapper with a facade to detect / prevent
>> +    components retaining references longer than they should.
> 
> On the plus side, the components are privileged, so if something goes wrong
> it's the component's fault and not a security issue. It depends on how much
> use this gets ultimately. I suppose quite a bit moving forward, so there
> should be an option there. It's obviously going to be cheap too.

Agreed. This is more of a protect the apps from themselves feature.

>> +New items for 10.0.x onwards:
>>
>> +1.  Remove APR connector.
>>
>> +2.  Remove org.apache.tomcat.jni and replace with the minimum necessary to
>> +    interface with OpenSSL and clones.
>>
> 
> Yes, but I still have no answer for the assertion that APR is still the
> fastest connector. So this performance drop has to be accepted. On the plus
> side, I would say APR users are not a majority, so this could go well.
> Functionally, APR is now more a problem than a solution in Tomcat
> ("sendfile" done differently works well now).

I thought the most recent round of testing showed that the difference
was marginal for "NIO HTTP" vs "APR HTTP" and "NIO+OpenSSL" vs "NIO APR
HTTPS".

If that isn't the case then there is much less of a case for removing APR.

> Ideas:
> - Someone mentioned a new compiler API could be used in Jasper
> - An embedded 2 API ?
> - Whatever from the early work of the EE EG (reactive shiny thingies ?)

All sound good.

Mark

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

Reply via email to