RFR: 8059570, Addition of tests for RowSetFactory and RowSetProvider
Lance Andersen
lance.andersen at oracle.com
Thu Oct 2 11:08:09 UTC 2014
Hi Daniel,
Thank you for you review.
On Oct 2, 2014, at 6:39 AM, Daniel Fuchs <daniel.fuchs at oracle.com> wrote:
> Hi Lance,
>
> I probably don't know enough about rowset to qualify as reviewer
> for this change, but I had a look at the tests - and I believe
> they look good.
Thank you.
> I wonder whether there should be some additional tests that would
> run with a security manager on?
There are tests elsewhere which run with a SecurityManager. I might add some here later but will be in a different test file
>
> The only thing unusual I noticed is that the jars seem to have a
> .DS_Store in their META-INF ;-)
thanks, deleted that. Have hidden files enabled on my mac and forgot to remove it.
Best
Lance
>
> best regards,
>
> -- daniel
>
> On 01/10/14 19:55, Lance Andersen wrote:
>> Hi all,
>>
>> Looking for a reviewer for the unit tests being added for RowSetFactory and RowSetProvider.
>>
>> The webrev can be found at http://cr.openjdk.java.net/~lancea/8059570/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
>>
>>
>>
>
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
More information about the core-libs-dev
mailing list