RFR: 8054555, javadoc clean-up in java.sql, javax.sql.*

[Ivan Gerasimov]

And a few more duplicates under jdk/src/share/classes/java/sql:

./NClob.java: * which *provides provides* methods for getting the
./Connection.java:     * @exception *SQLException SQLException* if a 
database access error occurs
./Connection.java:     * @exception *SQLException SQLException* if a 
database access error occurs
./Connection.java:     * @exception *SQLException SQLException* if a 
database access error occurs
./Connection.java:     * You must set *the the* values for the 
<code>TypeMap</code> prior to
./Connection.java:     * can *be be* determined by invoking
./DatabaseMetaData.java:     * <P>Each procedure description has *the 
the* following columns:
./DatabaseMetaData.java:     * <P>Each function description has *the 
the* following columns:

On 08.08.2014 4:09, Ivan Gerasimov wrote:
> Hi Lance!
>
> I like to grep for the word duplicates :)
> Under jdk/src/share/classes/javax/sql grep found:
>
> ./rowset/spi/SyncFactory.java:     * Adds*the the* given 
> synchronization provider to the factory register. Guidelines
> ./rowset/spi/SyncFactory.java:     * ROWSET properties and stores lazy 
> references into*the the* internal registry.
> ./rowset/spi/SyncProvider.java: * degrees *of of *care. The first 
> group of constants indicate how synchronization
> ./rowset/spi/package.html:For *the the* next level up, the
> ./rowset/BaseRowSet.java:     * in the driver documentation. Although 
> there are guidelines*for for *how
> ./rowset/RowSetProvider.java:     * in jars available to the runtime. 
> For example, to have*the the* RowSetFactory
> ./rowset/JoinRowSet.java: * A match column can *be be *set in two ways:
> ./rowset/JoinRowSet.java:     * An ANSI-style <code>JOIN</code> 
> providing *a a* full JOIN. Specifies that all
> ./rowset/serial/package.html:that *the the* originating data source 
> manages. <br>
> ./rowset/serial/package.html:<code>wasNull</code> is used to determine 
> whether *the the* last value read was SQL NULL.
> ./rowset/CachedRowSet.java:    * until*all all *components reading 
> this <code>CachedRowSet</code> object
> ./rowset/CachedRowSet.java:    *     row *in in* this 
> <code>CachedRowSet</code> object
> ./rowset/CachedRowSet.java:    * on *the the* rowset reader 
> (<code>RowSetReader</code> object) from the
> ./rowset/CachedRowSet.java:    *      was instantiated, or if none 
> *was was* set, the default provider
> ./rowset/FilteredRowSet.java: * <code>FilteredRowSet</code> object are 
> detected. A new filter*has has* an
> ./rowset/RowSetMetaDataImpl.java:     * Retrieves*the the* suggested 
> column title for the designated
> ./rowset/RowSetMetaDataImpl.java:         * The field that holds *the 
> the* type name used by this particular data source
> ./rowset/package.html:standard interfaces specified in this package 
> *and and* <b>may</b> extend the
>
> Would you please include these typo fixes in your patch?
>
> Sincerely yours,
> Ivan
>
> On 08.08.2014 3:57, Lance @ Oracle wrote:
>> Hi joe,
>>
>> Thank you for the quick review.
>>
>> I will change those to {@code Inputstream}
>>
>> 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
>> Sent from my iPad
>>
>>> On Aug 7, 2014, at 7:49 PM, Joe Darcy <joe.darcy at oracle.com> wrote:
>>>
>>>> On 08/07/2014 03:44 PM, Lance Andersen wrote:
>>>> Hi,
>>>>
>>>> Looking for a reviewer for corrections to various javadoc typos
>>>>
>>>> The webrev can be found at 
>>>> http://cr.openjdk.java.net/~lancea/8054555/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
>>>>
>>>>
>>>>
>>> Hi Lance,
>>>
>>> In src/share/classes/java/sql/CallableStatement.java
>>>
>>> @@ -1899,7 +1900,7 @@
>>>       * @param length the number of bytes in the parameter data.
>>>       * @throws SQLException  if parameterName does not correspond 
>>> to a named
>>>       * parameter; if the length specified
>>> -     * is less than zero; if the number of bytes in the inputstream 
>>> does not match
>>> +     * is less than zero; if the number of bytes in the Inputstream 
>>> does not match
>>>       * the specified length; if a database access error occurs or
>>>       * this method is called on a closed 
>>> <code>CallableStatement</code>
>>>       * @exception SQLFeatureNotSupportedException if the JDBC 
>>> driver does not support
>>>
>>> I think the new line would be better using either "input stream" or 
>>> "{@code InputStream}".
>>>
>>> Similar comment in src/share/classes/javax/sql/RowSet.java
>>>
>>> @@ -1420,7 +1420,7 @@
>>>    void setBlob (int i, Blob x) throws SQLException;
>>>
>>>    /**
>>> -     * Sets the designated parameter to a <code>InputStream</code> 
>>> object.  The inputstream must contain  the number
>>> +     * Sets the designated parameter to a <code>InputStream</code> 
>>> object.  The <code>InputStream</code> must contain  the number
>>>       * of characters specified by length otherwise a 
>>> <code>SQLException</code> will be
>>>       * generated when the <code>PreparedStatement</code> is executed.
>>>       * This method differs from the <code>setBinaryStream (int, 
>>> InputStream, int)</code>
>>>
>>> @@ -1474,7 +1474,8 @@
>>>          throws SQLException;
>>>
>>>    /**
>>> -     * Sets the designated parameter to a <code>InputStream</code> 
>>> object.  The <code>inputstream</code> must contain  the number
>>> +     * Sets the designated parameter to a <code>InputStream</code> 
>>> object.
>>> +     * The <code>inputstream</code> must contain the number
>>>       * of characters specified by length, otherwise a 
>>> <code>SQLException</code> will be
>>>       * generated when the <code>CallableStatement</code> is executed.
>>>       * This method differs from the <code>setBinaryStream (int, 
>>> InputStream, int)</code>
>>>
>>> Otherwise, the changes look okay.
>>>
>>> Cheers,
>>>
>>> -Joe
>>
>
>
>