-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27312/
-----------------------------------------------------------

(Updated Oct. 29, 2014, 9:22 p.m.)


Review request for Sqoop.


Changes
-------

Incorporating Veena'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/TestSeverity.java 
PRE-CREATION 
  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 
  shell/src/main/resources/shell-resource.properties 0e63c50 
  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

Reply via email to