-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27312/
-----------------------------------------------------------
(Updated Oct. 28, 2014, 10:20 p.m.)
Review request for Sqoop.
Changes
-------
Incorporated Abe's feedback.
Bugs: SQOOP-1469
https://issues.apache.org/jira/browse/SQOOP-1469
Repository: sqoop-sqoop2
Description
-------
This page is getting crazy big, but only contains the following change(s):
* Renamed class Status to Severity
* Renamed "FINE" to "OK"
* Renamed "ACCEPTABLE" to "WARNING"
* Renamed "UNACCEPTABLE" to "ERROR"
Which were suggestions that were there for quite some time.
Diffs (updated)
-----
client/src/main/java/org/apache/sqoop/client/SqoopClient.java 33a0c3c
common/src/main/java/org/apache/sqoop/json/ValidationResultBean.java 4b3fc37
common/src/main/java/org/apache/sqoop/model/MValidatedElement.java c0d678a
common/src/main/java/org/apache/sqoop/validation/ConfigValidationResult.java
4c4d123
common/src/main/java/org/apache/sqoop/validation/ConfigValidationRunner.java
8c66b3d
common/src/main/java/org/apache/sqoop/validation/Message.java 3361b6f
common/src/main/java/org/apache/sqoop/validation/Severity.java PRE-CREATION
common/src/main/java/org/apache/sqoop/validation/Status.java 5a546bf
common/src/main/java/org/apache/sqoop/validation/validators/AbstractValidator.java
9dbb44c
common/src/main/java/org/apache/sqoop/validation/validators/ClassAvailable.java
52a8fdd
common/src/main/java/org/apache/sqoop/validation/validators/Contains.java
8920c72
common/src/main/java/org/apache/sqoop/validation/validators/NotEmpty.java
248a2fa
common/src/main/java/org/apache/sqoop/validation/validators/NotNull.java
93b5fa7
common/src/main/java/org/apache/sqoop/validation/validators/NullOrContains.java
9d11a2e
common/src/main/java/org/apache/sqoop/validation/validators/StartsWith.java
808e1f7
common/src/test/java/org/apache/sqoop/json/TestValidationResultBean.java
bdbad72
common/src/test/java/org/apache/sqoop/model/TestMValidatedElement.java
a75a7cd
common/src/test/java/org/apache/sqoop/validation/TestStatus.java 99d8076
common/src/test/java/org/apache/sqoop/validation/TestValidationRunner.java
579d1c5
common/src/test/java/org/apache/sqoop/validation/validators/TestClassAvailable.java
3a15274
common/src/test/java/org/apache/sqoop/validation/validators/TestContains.java
e63a69a
common/src/test/java/org/apache/sqoop/validation/validators/TestNotEmpty.java
5c9169a
common/src/test/java/org/apache/sqoop/validation/validators/TestNotNull.java
91e5398
common/src/test/java/org/apache/sqoop/validation/validators/TestNullOrContains.java
88347eb
common/src/test/java/org/apache/sqoop/validation/validators/TestStartsWith.java
8c4f9e0
common/src/test/java/org/apache/sqoop/validation/validators/TestValidator.java
3f60ee6
connector/connector-generic-jdbc/src/main/java/org/apache/sqoop/connector/jdbc/configuration/FromJobConfig.java
12ceb21
connector/connector-generic-jdbc/src/main/java/org/apache/sqoop/connector/jdbc/configuration/LinkConfig.java
be86855
connector/connector-generic-jdbc/src/main/java/org/apache/sqoop/connector/jdbc/configuration/ToJobConfig.java
2428601
connector/connector-generic-jdbc/src/test/java/org/apache/sqoop/connector/jdbc/TestToInitializer.java
243de01
connector/connector-hdfs/src/main/java/org/apache/sqoop/connector/hdfs/configuration/LinkConfig.java
c63f8a8
connector/connector-hdfs/src/main/java/org/apache/sqoop/connector/hdfs/configuration/ToJobConfig.java
7fb1f74
connector/connector-hdfs/src/test/java/org/apache/sqoop/connector/hdfs/TestLinkConfig.java
176d0df
core/src/main/java/org/apache/sqoop/repository/Repository.java 79742b9
core/src/test/java/org/apache/sqoop/repository/TestJdbcRepository.java
a1e734e
server/src/main/java/org/apache/sqoop/handler/JobRequestHandler.java 5547988
server/src/main/java/org/apache/sqoop/handler/LinkRequestHandler.java 35a9635
shell/src/main/java/org/apache/sqoop/shell/CloneJobFunction.java 8188831
shell/src/main/java/org/apache/sqoop/shell/CloneLinkFunction.java c1a4f55
shell/src/main/java/org/apache/sqoop/shell/CreateJobFunction.java 6f136b2
shell/src/main/java/org/apache/sqoop/shell/CreateLinkFunction.java ce9988f
shell/src/main/java/org/apache/sqoop/shell/DeleteJobFunction.java da26153
shell/src/main/java/org/apache/sqoop/shell/DeleteLinkFunction.java c81917e
shell/src/main/java/org/apache/sqoop/shell/DisableJobFunction.java 9e46804
shell/src/main/java/org/apache/sqoop/shell/DisableLinkFunction.java 6b85292
shell/src/main/java/org/apache/sqoop/shell/EnableJobFunction.java 8575a84
shell/src/main/java/org/apache/sqoop/shell/EnableLinkFunction.java 1dd30e8
shell/src/main/java/org/apache/sqoop/shell/SetOptionFunction.java ccc067f
shell/src/main/java/org/apache/sqoop/shell/SetServerFunction.java 84df281
shell/src/main/java/org/apache/sqoop/shell/ShowConnectorFunction.java d605457
shell/src/main/java/org/apache/sqoop/shell/ShowDriverFunction.java 080792b
shell/src/main/java/org/apache/sqoop/shell/ShowJobFunction.java 0640283
shell/src/main/java/org/apache/sqoop/shell/ShowLinkFunction.java f4eae33
shell/src/main/java/org/apache/sqoop/shell/ShowOptionFunction.java 4bb0cab
shell/src/main/java/org/apache/sqoop/shell/ShowServerFunction.java 67eb6a6
shell/src/main/java/org/apache/sqoop/shell/ShowSubmissionFunction.java
2d00b88
shell/src/main/java/org/apache/sqoop/shell/ShowVersionFunction.java 3b4bb3f
shell/src/main/java/org/apache/sqoop/shell/StartJobFunction.java 4363f05
shell/src/main/java/org/apache/sqoop/shell/StatusJobFunction.java fb83af3
shell/src/main/java/org/apache/sqoop/shell/StopJobFunction.java 790c522
shell/src/main/java/org/apache/sqoop/shell/UpdateJobFunction.java dd075d7
shell/src/main/java/org/apache/sqoop/shell/UpdateLinkFunction.java 176833a
shell/src/main/java/org/apache/sqoop/shell/utils/ConfigDisplayer.java dea271a
shell/src/main/java/org/apache/sqoop/shell/utils/ConfigFiller.java c61d33b
test/src/main/java/org/apache/sqoop/test/testcases/ConnectorTestCase.java
af31769
tools/src/main/java/org/apache/sqoop/tools/tool/RepositoryLoadTool.java
64b08fc
Diff: https://reviews.apache.org/r/27312/diff/
Testing
-------
Unit tests are passing.
Thanks,
Jarek Cecho
