Dealing with DBCP exception

27 views
Skip to first unread message

Craig Jones

unread,
Jun 15, 2012, 9:46:50 PM6/15/12
to autopat...@googlegroups.com
To get AutoPatch 1.4.1 running, I had to clone the source code and add a catch to SqlUtil.close(,,) to deal wit the fact that DBCP's DelegatingResultSet acts badly:

            catch (AbstractMethodError e)
            {
                log.debug("AbstractMethodError closing ResultSet.  ResultSet might be a DelegatingResultSet with a badly implemented (i.e. missing) delegation to isClosed().", e);
            }

Should I fork the code and submit this with a pull request?

Scott Askew

unread,
Jun 16, 2012, 11:12:32 PM6/16/12
to autopat...@googlegroups.com
Hi Craig,

Nice catch. It would be great if you could fork the code and submit a pull request for this patch so we know that we've got the right fix in.

Thanks!

-scott
Reply all
Reply to author
Forward
0 new messages