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

[Lance Andersen - Oracle]

Thank you Ulf.

I deleted the extra line on 579

Best
Lance
On Jan 15, 2013, at 3:45 PM, Ulf Zibis wrote:

> 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
>> 
> 

-------------- next part --------------

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