RFR 8062198: Add RowSetMetaDataImpl Tests and add column range validation to isdefinitlyWritable
huizhe wang
huizhe.wang at oracle.com
Mon Oct 27 20:10:07 UTC 2014
Hi Lance,
Thanks for the explanation, so I learned a bit about these APIs :-) I
had thought RowSetMetaDataImpl is The implementation of a RowSet.
The changes look good to me.
-Joe
On 10/27/2014 12:37 PM, Lance Andersen wrote:
> Hi Joe,
>
> Thank you for taking time to help review.
> On Oct 27, 2014, at 3:27 PM, huizhe wang <huizhe.wang at oracle.com
> <mailto:huizhe.wang at oracle.com>> wrote:
>
>> Hi Lance,
>>
>> Does test37 needs to loop through all of the columns?
>
> Wanted a test which made sure that for the number of columns in the
> metadata that the fields can be set. Does it have to, not really, but
> it wanted to sanity check that they could
>> There doesn't seem to be a differentiator or different factor between
>> the columns, e.g if one column is set to auto_increment and another
>> not, the results would be different. Also in the test, some of the
>> setters seem to be invalid, for example, whether the column is
>> auto_increment or not shall affect setters such as setPrecision,
>> setScale, setNullable, isReadOnly, isWritable and etc., shouldn't it?
>
> For this abstract class it is just a matter of validating the methods
> can be set for the various columns.
>
> in an implementation of a RowSet, the data associated with a given
> column will have more actual meaning based on the column definition.
> So yes the values being set are somewhat artificial but that is OK
> in this instance.
>
> Also different databases for example can allow an auto-incrementable
> column to be writable, others do not so your milage varies as to which
> fields can and cannot have coresponding values.
>
> Best,
> Lance
>>
>> -Joe
>>
>> On 10/27/2014 11:06 AM, Lance Andersen wrote:
>>> Hi all,
>>>
>>> Need a reviewer for the RowSetMetaDataImpl tests which also
>>> includes a fix to isDefinitelyWritable so that it validates the
>>> column index ranges
>>>
>>> Included are some minor housekeeping changes to remove redundant classes
>>>
>>> The webrev can be found at
>>> http://cr.openjdk.java.net/~lancea/8062198/webrev.00/
>>> <http://cr.openjdk.java.net/%7Elancea/8062198/webrev.00/>
>>>
>>> Best
>>> Lance
>>>
>>>
>>> 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>
>>>
>>>
>>>
>>
>
> <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>
> <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