review request 8006139, add missing methods to javax.sql.rowset.serial.SQLInput/OutputImpl

[Ulf Zibis]

Looks great!

Little nit:
SQLOutputImpl.java line 579 could be dropped.

-Ulf


Am 15.01.2013 17:48, schrieb Lance Andersen - Oracle:
> Here is a revision
>
> http://cr.openjdk.java.net/~lancea/8006139/webrev.01
>
> I still have to enter the JBS entry for the javadoc clarifications (and I also found another javadoc issue that was due to incorrect cut & paste when the code was written) and ccc request
>
> As i mentioned in an earlier thread, these classes are hardly ever, if at all used and would only be used when UDTs are used and the majority of databases do not support this.
>
>
> Best
> lance
> On Jan 14, 2013, at 5:11 AM, Alan Bateman wrote:
>
>> On 13/01/2013 23:51, Lance Andersen - Oracle wrote:
>>> :
>>>
>>>>> One other thing is that the CCE has a side-effect in that it "consumes" the next attribute. The methods could be changed to peek at the next attribute but that wouldn't work without synchronization or making it clear in the spec that the it is not a thread-safe implementation of SQLInput.
>>> I really want to keep the changes to the bare minimum here as this and the other serial classes are hardly, if ever used at all.
>> I understand, but if you add a catch-all in the class description to cover the CCE case then this could be part of the same paragraph.
>>
>> -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
>