2014-08-12 0:38 GMT+04:00 <ma...@apache.org>:
> Author: markt
> Date: Mon Aug 11 20:38:18 2014
> New Revision: 1617359
>
> URL: http://svn.apache.org/r1617359
> Log:
> Silence some unnecessary nags
>
> Modified:
>
> tomcat/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/interceptor/StatementFinalizer.java
>
> Modified:
> tomcat/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/interceptor/StatementFinalizer.java
> URL:
> http://svn.apache.org/viewvc/tomcat/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/interceptor/StatementFinalizer.java?rev=1617359&r1=1617358&r2=1617359&view=diff
> ==============================================================================
> ---
> tomcat/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/interceptor/StatementFinalizer.java
> (original)
> +++
> tomcat/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/interceptor/StatementFinalizer.java
> Mon Aug 11 20:38:18 2014
> @@ -52,6 +52,7 @@ public class StatementFinalizer extends
> return statement;
> }
>
> + @SuppressWarnings("null") // st is not null when used
Where it is used?
What guarantees that WeakReference to that Statement does not return
null? Where is that hard reference that prevents the statement from
being garbage collected?
With the current code I think ws.getStatement() can return null.
I am starting to think that we do not need a WeakReference for a
Statement, but a usual hard reference would work better.
(Just saying, without digging into the code. The relevant questions are
- Can it be that the only hard reference to Statement object are in user's code?
- If yes, then what happens if user clears those references and allows
the Statement to be GC'ed? Are java.sql.Statement implementations
expected to implement finalize() method that calls close() on them?
)
> @Override
> public void closeInvoked() {
> while (statements.size()>0) {
>
Best regards,
Konstantin Kolinko
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
