-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/7409/
-----------------------------------------------------------
Review request for Sqoop.
Description
-------
This patch removes cca 200 FindBugs warning and errors.
This addresses bug SQOOP-618.
https://issues.apache.org/jira/browse/SQOOP-618
Diffs
-----
client/src/main/java/org/apache/sqoop/client/core/Environment.java b496d43
client/src/main/java/org/apache/sqoop/client/shell/SqoopShell.java 91682d1
client/src/main/java/org/apache/sqoop/client/utils/FormDisplayer.java 02588c7
client/src/main/java/org/apache/sqoop/client/utils/FormFiller.java af3ff8a
common/src/main/java/org/apache/sqoop/common/ErrorCode.java f53d972
common/src/main/java/org/apache/sqoop/common/SqoopProtocolConstants.java
b06e059
common/src/main/java/org/apache/sqoop/common/VersionInfo.java 02f8cf3
common/src/main/java/org/apache/sqoop/json/JsonBean.java aa84935
common/src/main/java/org/apache/sqoop/json/ValidationBean.java b56f6ed
common/src/main/java/org/apache/sqoop/json/util/FormSerialization.java
6b7332a
common/src/main/java/org/apache/sqoop/json/util/ResourceBundleSerialization.java
dc4aac3
common/src/main/java/org/apache/sqoop/model/MConnection.java 97771a3
common/src/main/java/org/apache/sqoop/model/MJob.java 186716a
common/src/main/java/org/apache/sqoop/model/MMapInput.java 9818d71
common/src/main/java/org/apache/sqoop/model/MValidatedElement.java a0c5074
connector/connector-generic-jdbc/src/main/java/org/apache/sqoop/connector/jdbc/GenericJdbcConnector.java
4363e0a
core/src/main/java/org/apache/sqoop/connector/ConnectorManager.java ea368da
core/src/main/java/org/apache/sqoop/core/ConfigurationListener.java e106043
core/src/main/java/org/apache/sqoop/core/ConfigurationProvider.java 3dcb853
core/src/main/java/org/apache/sqoop/core/PropertiesConfigurationProvider.java
8837dc9
core/src/main/java/org/apache/sqoop/core/SqoopConfiguration.java 3d84d4a
core/src/main/java/org/apache/sqoop/framework/FrameworkConstants.java eb6e8b2
core/src/main/java/org/apache/sqoop/framework/FrameworkManager.java 657eab2
core/src/main/java/org/apache/sqoop/job/etl/HdfsSequenceImportLoader.java
ad513e1
core/src/main/java/org/apache/sqoop/job/etl/HdfsTextImportLoader.java 1368a5e
core/src/main/java/org/apache/sqoop/repository/JdbcRepository.java be23ebf
core/src/main/java/org/apache/sqoop/repository/JdbcRepositoryContext.java
dc01d89
core/src/main/java/org/apache/sqoop/repository/JdbcRepositoryHandler.java
aaca7f7
core/src/main/java/org/apache/sqoop/repository/JdbcRepositoryTransaction.java
96c75be
core/src/main/java/org/apache/sqoop/repository/Repository.java 5580b4e
core/src/main/java/org/apache/sqoop/repository/RepositoryManager.java 17c81a1
core/src/main/java/org/apache/sqoop/repository/RepositoryProvider.java
7326536
core/src/main/java/org/apache/sqoop/repository/RepositoryTransaction.java
d86d79b
repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbyRepositoryHandler.java
17e43b8
server/src/main/java/org/apache/sqoop/handler/ConnectionRequestHandler.java
49c93b4
server/src/main/java/org/apache/sqoop/handler/ConnectorRequestHandler.java
b8dc3b8
server/src/main/java/org/apache/sqoop/handler/FrameworkRequestHandler.java
9a60c63
server/src/main/java/org/apache/sqoop/handler/JobRequestHandler.java 6fbf47c
server/src/main/java/org/apache/sqoop/handler/VersionRequestHandler.java
75c36d8
server/src/main/java/org/apache/sqoop/server/RequestContext.java 90e174e
server/src/main/java/org/apache/sqoop/server/RequestHandler.java 43d98c5
server/src/main/java/org/apache/sqoop/server/ServerInitializer.java cd2b13e
spi/src/main/java/org/apache/sqoop/connector/spi/SqoopConnector.java b452bc3
spi/src/main/java/org/apache/sqoop/job/etl/Context.java 7256281
spi/src/main/java/org/apache/sqoop/job/etl/MutableContext.java 2bfe89f
Diff: https://reviews.apache.org/r/7409/diff/
Testing
-------
* mvn test
* Manual CRUD test for connection and job objects
Thanks,
Jarek Cecho
