KeiichiFujino commented on issue #140: jdbc-pool: Improve maxAge handling
URL: https://github.com/apache/tomcat/pull/140#issuecomment-473176479
 
 
   > I fixed the missing calls to PooledConnection#validate(int).
   
   Thanks.
   
   
   
   > Looking at the code it seems that the timestamp should only be updated 
when user-initiated (like borrow or return) operations are executed on the 
PooledConnection, not when (possibly internal) operations are executed on the 
underlying JDBC Connection - right ?
   
   That's right.
   There is no need to call PooledConnection#setTimestamp(long) when 
reconnecting an idle connection.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

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

Reply via email to