RFR: 8054555, javadoc clean-up in java.sql, javax.sql.*
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@oracle.com
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@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
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@oracle.com Sent from my iPad
On Aug 7, 2014, at 7:49 PM, Joe Darcy <joe.darcy@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@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
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@oracle.com Sent from my iPad
On Aug 7, 2014, at 7:49 PM, Joe Darcy <joe.darcy@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@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
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@oracle.com Sent from my iPad
On Aug 7, 2014, at 7:49 PM, Joe Darcy <joe.darcy@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@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
Hi Joe, Ivan, Thank you for the feedback. Attached is the revised webrev http://cr.openjdk.java.net/~lancea/8054555/webrev.01/ Best, Lance On Aug 7, 2014, at 8:16 PM, Ivan Gerasimov <ivan.gerasimov@oracle.com> wrote:
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@oracle.com Sent from my iPad
On Aug 7, 2014, at 7:49 PM, Joe Darcy <joe.darcy@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@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
Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 Lance.Andersen@oracle.com
Hi Lance! 1) src/share/classes/java/sql/CallableStatement.java + * is less than zero; if the number of bytes in the "{@code InputStream} should be + * is less than zero; if the number of bytes in the {@code InputStream} 3) src/share/classes/javax/sql/RowSet.java + * The <code>reader</code> must contain the number should the Reader be capitalized as in the line above it? + * The reader must contain the number should the reader be marked with {@code} for consistency? 4) src/share/classes/javax/sql/rowset/serial/SerialBlob.java + // is for backwards compatibiiity should be + // is for backwards compatibility Otherwise look good! Sincerely yours, Ivan On 08.08.2014 15:07, Lance Andersen wrote:
Hi Joe, Ivan,
Thank you for the feedback.
Attached is the revised webrev
http://cr.openjdk.java.net/~lancea/8054555/webrev.01/ <http://cr.openjdk.java.net/%7Elancea/8054555/webrev.01/>
Best, Lance On Aug 7, 2014, at 8:16 PM, Ivan Gerasimov <ivan.gerasimov@oracle.com <mailto:ivan.gerasimov@oracle.com>> wrote:
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@oracle.com Sent from my iPad
On Aug 7, 2014, at 7:49 PM, Joe Darcy <joe.darcy@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@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
<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@oracle.com <mailto:Lance.Andersen@oracle.com>
Hi Ivan, Here is the revised with your additional input. I think I needed extra coffee this morning http://cr.openjdk.java.net/~lancea/8054555/webrev.02/ Best, Lance On Aug 8, 2014, at 9:39 AM, Ivan Gerasimov <ivan.gerasimov@oracle.com> wrote:
Hi Lance!
1) src/share/classes/java/sql/CallableStatement.java
+ * is less than zero; if the number of bytes in the "{@code InputStream} should be + * is less than zero; if the number of bytes in the {@code InputStream}
3) src/share/classes/javax/sql/RowSet.java
+ * The <code>reader</code> must contain the number should the Reader be capitalized as in the line above it?
+ * The reader must contain the number should the reader be marked with {@code} for consistency?
4) src/share/classes/javax/sql/rowset/serial/SerialBlob.java
+ // is for backwards compatibiiity should be + // is for backwards compatibility
Otherwise look good!
Sincerely yours, Ivan
On 08.08.2014 15:07, Lance Andersen wrote:
Hi Joe, Ivan,
Thank you for the feedback.
Attached is the revised webrev
http://cr.openjdk.java.net/~lancea/8054555/webrev.01/
Best, Lance On Aug 7, 2014, at 8:16 PM, Ivan Gerasimov <ivan.gerasimov@oracle.com> wrote:
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@oracle.com Sent from my iPad
On Aug 7, 2014, at 7:49 PM, Joe Darcy <joe.darcy@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@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
<Mail Attachment.gif>
Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 Lance.Andersen@oracle.com
Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 Lance.Andersen@oracle.com
Looks good! I've scanned through the webrev and found no issues. I'm not an official reviewer though. Sincerely yours, Ivan On 08.08.2014 21:14, Lance Andersen wrote:
Hi Ivan,
Here is the revised with your additional input. I think I needed extra coffee this morning http://cr.openjdk.java.net/~lancea/8054555/webrev.02/ <http://cr.openjdk.java.net/%7Elancea/8054555/webrev.02/>
Best, Lance On Aug 8, 2014, at 9:39 AM, Ivan Gerasimov <ivan.gerasimov@oracle.com <mailto:ivan.gerasimov@oracle.com>> wrote:
Hi Lance!
1) src/share/classes/java/sql/CallableStatement.java
+ * is less than zero; if the number of bytes in the "{@code InputStream} should be + * is less than zero; if the number of bytes in the {@code InputStream}
3) src/share/classes/javax/sql/RowSet.java
+ * The <code>reader</code> must contain the number should the Reader be capitalized as in the line above it?
+ * The reader must contain the number should the reader be marked with {@code} for consistency?
4) src/share/classes/javax/sql/rowset/serial/SerialBlob.java
+ // is for backwards compatibiiity should be + // is for backwards compatibility
Otherwise look good!
Sincerely yours, Ivan
On 08.08.2014 15:07, Lance Andersen wrote:
Hi Joe, Ivan,
Thank you for the feedback.
Attached is the revised webrev
http://cr.openjdk.java.net/~lancea/8054555/webrev.01/ <http://cr.openjdk.java.net/%7Elancea/8054555/webrev.01/>
Best, Lance On Aug 7, 2014, at 8:16 PM, Ivan Gerasimov <ivan.gerasimov@oracle.com <mailto:ivan.gerasimov@oracle.com>> wrote:
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@oracle.com Sent from my iPad
> On Aug 7, 2014, at 7:49 PM, Joe Darcy <joe.darcy@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@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
<Mail Attachment.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> <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@oracle.com <mailto:Lance.Andersen@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@oracle.com <mailto:Lance.Andersen@oracle.com>
participants (4)
-
Ivan Gerasimov
-
Joe Darcy
-
Lance ＠ Oracle
-
Lance Andersen