Review request for 7192302 Remove JDBCRowSetImpl dependency on java.beans

[Lance Andersen - Oracle]

On Sep 5, 2012, at 5:17 PM, Alan Bateman wrote:

> On 05/09/2012 22:04, Lance Andersen - Oracle wrote:
>> Hi all,
>> 
>> Looking for a reviewer for the removal of PropertyChangeSupport from JDBCRowSetImpl that was originally going to be used by the EOL Rave product.  As it is no longer needed the code has been removed.  The SQE and RowSet TCK tests all continue to run without regression.
>> 
>> The webrev can be found at http://cr.openjdk.java.net/~lancea/7192302/webrev.00
>> 
> Thanks Lance, it's good to remove this dependency. In both commit and rollback then it looks to me that the setting of oldVal can be removed. Otherwise looks good to me.

Yes your right.  setType has some similar crud.  I will make a quick swing through and nuke that also as I should have done it as part of the pass.

Best
lance
> 
> -Alan


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