Fix Javadocs

spring-batch-infrastructure/src/main/java/org/springframework/batch/item/database/support/AbstractSqlPagingQueryProvider.java

@@ -15,22 +15,19 @@
  */
 package org.springframework.batch.item.database.support;
 
-import org.springframework.util.Assert;
-import org.springframework.jdbc.support.MetaDataAccessException;
-import org.springframework.jdbc.support.JdbcUtils;
-import org.springframework.dao.InvalidDataAccessResourceUsageException;
-
 import javax.sql.DataSource;
 
+import org.springframework.util.Assert;
+
 /**
  * Abstract SQL Paging Query Provider to serve as a base class for all provided SQL paging query providers.
  *
  * Any implementation must provide a way to specify the select clause, from clause and optionally a where clause. 
  * In addition a way to specify a single column sort key must also be provided. This sort key will be used to 
- * provide the paging functinality. It is recommended that there should be an index for the sort key to provide 
+ * provide the paging functionality. It is recommended that there should be an index for the sort key to provide 
  * better performance.
  *
- * Provides properties and preperation for the mandatory "selectClause" and "fromClause" as well as for the
+ * Provides properties and preparation for the mandatory "selectClause" and "fromClause" as well as for the
  * optional "whereClause".  Also provides property for the mandatory "sortKey".
  *
  * @author Thomas Risberg
@@ -84,7 +81,7 @@ public abstract class AbstractSqlPagingQueryProvider implements PagingQueryProvi
 
 	/**
 	 *
-	 * @return
+	 * @return SQL "from" clause
 	 */
 	protected String getFromClause() {
 		return fromClause;
@@ -106,7 +103,7 @@ public abstract class AbstractSqlPagingQueryProvider implements PagingQueryProvi
 
 	/**
 	 *
-	 * @return
+	 * @return WHERE clause part of SQL query string
 	 */
 	protected String getWhereClause() {
 		return whereClause;
@@ -121,7 +118,7 @@ public abstract class AbstractSqlPagingQueryProvider implements PagingQueryProvi
 
 	/**
 	 *
-	 * @return
+	 * @return sortKey key to use to sort and limit page content
 	 */
 	protected String getSortKey() {
 		return sortKey;

spring-batch-infrastructure/src/main/java/org/springframework/batch/item/database/support/OraclePagingQueryProvider.java

@@ -1,10 +1,5 @@
 package org.springframework.batch.item.database.support;
 
-import org.springframework.jdbc.support.JdbcUtils;
-import org.springframework.dao.InvalidDataAccessResourceUsageException;
-
-import javax.sql.DataSource;
-
 /**
  * Oracle implementation of a  {@link org.springframework.batch.item.database.support.PagingQueryProvider} using
  * database specific features.

spring-batch-infrastructure/src/test/java/org/springframework/batch/item/database/support/DerbyPagingQueryProviderTests.java

@@ -27,7 +27,6 @@ import org.springframework.dao.InvalidDataAccessResourceUsageException;
 
 import javax.sql.DataSource;
 import java.sql.DatabaseMetaData;
-import java.sql.SQLException;
 import java.sql.Connection;
 
 /**

spring-batch-infrastructure/src/test/java/org/springframework/batch/item/database/support/SqlWindowingPagingQueryProviderTests.java

@@ -18,7 +18,6 @@ package org.springframework.batch.item.database.support;
 import static org.junit.Assert.*;
 
 import org.junit.Test;
-import org.junit.Before;
 
 /**
  * @author Thomas Risberg