Hi Dave,
thanks for the feedback. Adding @Deprecated changes the signatures so I need
to coordinate any changes as it will result in TCK signature failures. This is
something I will most likely do as part of the JDBC 4.2 work after giving the
JDBC EG a chance for input.
There are several places in RowSet that also require the use of @Deprecated:
---------------------------------------------------
/Users/lanceandersen/NetBeansProjects/JDBC4.2/jdbc4.0/src/javax/sql/rowset/BaseRowSet.java:2393:
warning: [dep-ann] deprecated item is not annotated with @Deprecated
public void setUnicodeStream(int parameterIndex, java.io.InputStream x, int
length) throws SQLException {
/Users/lanceandersen/NetBeansProjects/JDBC4.2/jdbc4.0/src/com/sun/rowset/CachedRowSetImpl.java:2043:
warning: [dep-ann] deprecated item is not annotated with @Deprecated
public BigDecimal getBigDecimal(int columnIndex, int scale) throws
SQLException {
/Users/lanceandersen/NetBeansProjects/JDBC4.2/jdbc4.0/src/com/sun/rowset/CachedRowSetImpl.java:2378:
warning: [dep-ann] deprecated item is not annotated with @Deprecated
public java.io.InputStream getUnicodeStream(int columnIndex) throws
SQLException {
/Users/lanceandersen/NetBeansProjects/JDBC4.2/jdbc4.0/src/com/sun/rowset/CachedRowSetImpl.java:2647:
warning: [dep-ann] deprecated item is not annotated with @Deprecated
public BigDecimal getBigDecimal(String columnName, int scale) throws
SQLException {
/Users/lanceandersen/NetBeansProjects/JDBC4.2/jdbc4.0/src/com/sun/rowset/CachedRowSetImpl.java:2778:
warning: [dep-ann] deprecated item is not annotated with @Deprecated
public java.io.InputStream getUnicodeStream(String columnName) throws
SQLException {
/Users/lanceandersen/NetBeansProjects/JDBC4.2/jdbc4.0/src/java/sql/ResultSet.java:361:
warning: [dep-ann] deprecated item is not annotated with @Deprecated
BigDecimal getBigDecimal(int columnIndex, int scale) throws SQLException;
/Users/lanceandersen/NetBeansProjects/JDBC4.2/jdbc4.0/src/java/sql/ResultSet.java:480:
warning: [dep-ann] deprecated item is not annotated with @Deprecated
java.io.InputStream getUnicodeStream(int columnIndex) throws SQLException;
/Users/lanceandersen/NetBeansProjects/JDBC4.2/jdbc4.0/src/java/sql/ResultSet.java:646:
warning: [dep-ann] deprecated item is not annotated with @Deprecated
BigDecimal getBigDecimal(String columnLabel, int scale) throws SQLException;
/Users/lanceandersen/NetBeansProjects/JDBC4.2/jdbc4.0/src/java/sql/ResultSet.java:763:
warning: [dep-ann] deprecated item is not annotated with @Deprecated
java.io.InputStream getUnicodeStream(String columnLabel) throws
SQLException;
/Users/lanceandersen/NetBeansProjects/JDBC4.2/jdbc4.0/src/java/sql/Date.java:54:
warning: [dep-ann] deprecated item is not annotated with @Deprecated
public Date(int year, int month, int day) {
/Users/lanceandersen/NetBeansProjects/JDBC4.2/jdbc4.0/src/java/sql/Date.java:182:
warning: [dep-ann] deprecated item is not annotated with @Deprecated
public int getHours() {
/Users/lanceandersen/NetBeansProjects/JDBC4.2/jdbc4.0/src/java/sql/Date.java:194:
warning: [dep-ann] deprecated item is not annotated with @Deprecated
public int getMinutes() {
/Users/lanceandersen/NetBeansProjects/JDBC4.2/jdbc4.0/src/java/sql/Date.java:206:
warning: [dep-ann] deprecated item is not annotated with @Deprecated
public int getSeconds() {
/Users/lanceandersen/NetBeansProjects/JDBC4.2/jdbc4.0/src/java/sql/Date.java:218:
warning: [dep-ann] deprecated item is not annotated with @Deprecated
public void setHours(int i) {
/Users/lanceandersen/NetBeansProjects/JDBC4.2/jdbc4.0/src/java/sql/Date.java:230:
warning: [dep-ann] deprecated item is not annotated with @Deprecated
public void setMinutes(int i) {
/Users/lanceandersen/NetBeansProjects/JDBC4.2/jdbc4.0/src/java/sql/Date.java:242:
warning: [dep-ann] deprecated item is not annotated with @Deprecated
public void setSeconds(int i) {
/Users/lanceandersen/NetBeansProjects/JDBC4.2/jdbc4.0/src/java/sql/PreparedStatement.java:347:
warning: [dep-ann] deprecated item is not annotated with @Deprecated
void setUnicodeStream(int parameterIndex, java.io.InputStream x,
/Users/lanceandersen/NetBeansProjects/JDBC4.2/jdbc4.0/src/java/sql/CallableStatement.java:298:
warning: [dep-ann] deprecated item is not annotated with @Deprecated
BigDecimal getBigDecimal(int parameterIndex, int scale)
/Users/lanceandersen/NetBeansProjects/JDBC4.2/jdbc4.0/src/com/sun/rowset/JdbcRowSetImpl.java:1039:
warning: [dep-ann] deprecated item is not annotated with @Deprecated
public BigDecimal getBigDecimal(int columnIndex, int scale) throws
SQLException {
/Users/lanceandersen/NetBeansProjects/JDBC4.2/jdbc4.0/src/com/sun/rowset/JdbcRowSetImpl.java:1177:
warning: [dep-ann] deprecated item is not annotated with @Deprecated
public java.io.InputStream getUnicodeStream(int columnIndex) throws
SQLException {
/Users/lanceandersen/NetBeansProjects/JDBC4.2/jdbc4.0/src/com/sun/rowset/JdbcRowSetImpl.java:1359:
warning: [dep-ann] deprecated item is not annotated with @Deprecated
public BigDecimal getBigDecimal(String columnName, int scale) throws
SQLException {
/Users/lanceandersen/NetBeansProjects/JDBC4.2/jdbc4.0/src/com/sun/rowset/JdbcRowSetImpl.java:1484:
warning: [dep-ann] deprecated item is not annotated with @Deprecated
public java.io.InputStream getUnicodeStream(String columnName) throws
SQLException {
/Users/lanceandersen/NetBeansProjects/JDBC4.2/jdbc4.0/src/com/sun/rowset/JoinRowSetImpl.java:1156:
warning: [dep-ann] deprecated item is not annotated with @Deprecated
public BigDecimal getBigDecimal(int columnIndex, int scale) throws
SQLException {
/Users/lanceandersen/NetBeansProjects/JDBC4.2/jdbc4.0/src/com/sun/rowset/JoinRowSetImpl.java:1267:
warning: [dep-ann] deprecated item is not annotated with @Deprecated
public java.io.InputStream getUnicodeStream(int columnIndex) throws
SQLException {
/Users/lanceandersen/NetBeansProjects/JDBC4.2/jdbc4.0/src/com/sun/rowset/JoinRowSetImpl.java:1439:
warning: [dep-ann] deprecated item is not annotated with @Deprecated
public BigDecimal getBigDecimal(String columnName, int scale) throws
SQLException {
/Users/lanceandersen/NetBeansProjects/JDBC4.2/jdbc4.0/src/com/sun/rowset/JoinRowSetImpl.java:1555:
warning: [dep-ann] deprecated item is not annotated with @Deprecated
public java.io.InputStream getUnicodeStream(String columnName) throws
SQLException {
/Users/lanceandersen/NetBeansProjects/JDBC4.2/jdbc4.0/src/com/sun/rowset/internal/SyncResolverImpl.java:1291:
warning: [dep-ann] deprecated item is not annotated with @Deprecated
public BigDecimal getBigDecimal(int columnIndex, int scale) throws
SQLException {
/Users/lanceandersen/NetBeansProjects/JDBC4.2/jdbc4.0/src/com/sun/rowset/internal/SyncResolverImpl.java:1427:
warning: [dep-ann] deprecated item is not annotated with @Deprecated
public java.io.InputStream getUnicodeStream(int columnIndex) throws
SQLException {
/Users/lanceandersen/NetBeansProjects/JDBC4.2/jdbc4.0/src/com/sun/rowset/internal/SyncResolverImpl.java:1656:
warning: [dep-ann] deprecated item is not annotated with @Deprecated
public BigDecimal getBigDecimal(String columnName, int scale) throws
SQLException {
/Users/lanceandersen/NetBeansProjects/JDBC4.2/jdbc4.0/src/com/sun/rowset/internal/SyncResolverImpl.java:1787:
warning: [dep-ann] deprecated item is not annotated with @Deprecated
public java.io.InputStream getUnicodeStream(String columnName) throws
SQLException {
/Users/lanceandersen/NetBeansProjects/JDBC4.2/jdbc4.0/src/java/sql/DriverManager.java:433:
warning: [dep-ann] deprecated item is not annotated with @Deprecated
public static void setLogStream(java.io.PrintStream out) {
/Users/lanceandersen/NetBeansProjects/JDBC4.2/jdbc4.0/src/java/sql/DriverManager.java:455:
warning: [dep-ann] deprecated item is not annotated with @Deprecated
public static java.io.PrintStream getLogStream() {
---------------------------------------------------
I could probably add the annotations to the com/sun/rowset/* classes now as
they are the RowSet RI, not public facing interfaces. Alan, Stuart your
thoughts?
I can add the @Override in DriverManager while I am there, but do not want to
make a pass through the rest of the code as part of this putback to do it
elsewhere as I would like to get these back.
Again, thank you for your feedback.
Best
Lance
On Dec 2, 2011, at 6:12 AM, David Schlosnagle wrote:
> On Thu, Dec 1, 2011 at 10:08 PM, Lance Andersen - Oracle
> <[email protected]> wrote:> Sadly, once i start digging, i
> found more changes :-(>> Here are the latest revisions with the
> additions/changes since the last review,
> http://cr.openjdk.java.net/~lancea/7116445/webrev.03:
> Lance,
>
> I'd suggest also including the attached changes [1] to your current
> webrev to remove the remaining warnings from java.sql.*.
>
> [1]: http://dl.dropbox.com/u/23832908/openjdk/7116445-additions.patch
>
> Thanks,
> Dave
Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering
1 Network Drive
Burlington, MA 01803
[email protected]
