Review needed: 8004374 : Fwd: JDBC bug: Incorrect number of conflicts is reported by CachedRowSetWriter.writeData (original) (raw)

Frank Ding dingxmin at linux.vnet.ibm.com
Mon Dec 10 07:44:33 UTC 2012

Hi Lance, The code refactory looks good. By the way, the newly added unit test is not jtreg test case?

Best regards, Frank

On 12/5/2012 4:38 AM, Lance Andersen - Oracle wrote:

All,

Attached is the patch for: 8004374 based off the issue that Frank reported. for http://cr.openjdk.java.net/~lancea/8004374/webrev.00/ <http://cr.openjdk.java.net/%7Elancea/8004374/webrev.00/> The TCK, SQE and the JDBC Unit Tests run clean. I added a new Unit Test to validate the issue. Frank, I did not use your fix as I was able to clean the code up a bit more and get rid of more crud while addressing it. It is similar though. Best Lance <http://oracle.com/us/design/oracle-email-sig-198324.gif>

<http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>

More information about the core-libs-dev mailing list