Hi Francesco,

Am 30.05.2018 um 11:11 schrieb Francesco Chicchiriccò:
On 29/05/2018 20:36, Guido Wimmel wrote:
Hi,

the recommended JavaEE configuration example on http://syncope.apache.org/docs/reference-guide.html#javaee-container contains

<Resource ... removeAbandonedTimeout="20000" ... />

Is this intentional? Seems strange to me, as this setting is in seconds (corresponding to more than 5 hours), according to https://tomcat.apache.org/tomcat-8.5-doc/jdbc-pool.html

Seems to have been introduced long ago in https://issues.apache.org/jira/browse/SYNCOPE-61 , which talks about milliseconds.

Or do I overlook something here?

Hi Guido,
it seems you're right.

I would say that we should replace 20000 with 20.
(I must admit that since when we introduced HikariCP I am not regularly using the Tomcat DBCP, not even in production).

Would you like to take care of such change both in

https://cwiki.apache.org/confluence/display/SYNCOPE/Run+Syncope+in+real+environments#RunSyncopeinrealenvironments-context.xml

and

https://github.com/apache/syncope/blob/2_0_X/src/main/asciidoc/reference-guide/workingwithapachesyncope/systemadministration/javaeecontainer.adoc

?
Regards.

Sure, I can make the changes.
Default in Tomcat is 60. 20 might also be appropriate as a recommendation if there are no long running queries.

To be sure: the change in git should really be in the 2_0_X branch of https://git-wip-us.apache.org/repos/asf/syncope.git , right? Any requirements on the commit message?
For the Wiki change I'd need change permission (Username: wimmelg).

Cheers, Guido



Reply via email to