Review request for 7192302 Remove JDBCRowSetImpl dependency on java.beans

Alan Bateman Alan.Bateman at
Thu Sep 6 13:31:09 UTC 2012

On 06/09/2012 14:09, Lance Andersen - Oracle wrote:
> :
>> The latest webrev looks okay except that in one of the constructors 
>> you have removed a call to ensure that the connection is established, 
>> I'm not sure about the significance of that.
> This is not needed here and given I have already tested with this 
> removed, I figured I would OK to keep this as part of the change
Okay, I'll have to trust you on this one but I am a little bit concerned 
that it could cause a NPE, say someone creates a JdbcRowSet and invokes 
a method such as comment, rollback or getAutoCommit without doing an 
explicit connect.

> I left those in as a reminder to go back as part of the rest of the 
> Rave clean-up.  I would prefer to leave them for now and when I  make 
> another pass for Rave, I will get rid of them

>> One method that looks like it could be removed too is setConcurrency 
>> but I agree that keeping focused on just removing the beans 
>> dependency is right for now.
>  I had thought about that but I have to think about this one a bit 
> more as the getConcurrency() is leveraging the value returned from the 
> active ResultSet which is why I did not remove this at this time.

Okay although it looks to be just an optimization introduced as part of 
adding listener events.  I'm fine with leaving it as it is.


More information about the core-libs-dev mailing list