----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8305/ -----------------------------------------------------------
(Updated Dec. 5, 2012, 6:41 p.m.) Review request for Sqoop and Jarek Cecho. Changes ------- Changes from v4 Fixed ClientError - Removed ResourceBundle import and fixed the client message to be final. Some issue with patch I have to look at. The same issue was between v2 and v3 also that I noticed. Thanks for catching that Fixed the version message for client and server to be a single one Description ------- I have moved localizable strings to the client resources (those that are descriptions, messages in general etc). Also consolidated constants to one place and removed repetitive occurrences. 4 more files in utils need to be updated, but wanted to get this reviewed and take that after this Diffs (updated) ----- client/src/main/java/org/apache/sqoop/client/core/Constants.java 47c0547 client/src/main/java/org/apache/sqoop/client/shell/CloneCommand.java 847a6ad client/src/main/java/org/apache/sqoop/client/shell/CloneConnectionFunction.java 21c41aa client/src/main/java/org/apache/sqoop/client/shell/CloneJobFunction.java b0e8d90 client/src/main/java/org/apache/sqoop/client/shell/CreateCommand.java 2453543 client/src/main/java/org/apache/sqoop/client/shell/CreateConnectionFunction.java 734276d client/src/main/java/org/apache/sqoop/client/shell/CreateJobFunction.java 0b685bf client/src/main/java/org/apache/sqoop/client/shell/DeleteCommand.java bb09bf3 client/src/main/java/org/apache/sqoop/client/shell/DeleteConnectionFunction.java ee2a1cf client/src/main/java/org/apache/sqoop/client/shell/DeleteJobFunction.java acc8e21 client/src/main/java/org/apache/sqoop/client/shell/HelpCommand.java 03499d8 client/src/main/java/org/apache/sqoop/client/shell/SetCommand.java 20c8090 client/src/main/java/org/apache/sqoop/client/shell/SetOptionFunction.java 3764306 client/src/main/java/org/apache/sqoop/client/shell/SetServerFunction.java daf1ff4 client/src/main/java/org/apache/sqoop/client/shell/ShowCommand.java bd74253 client/src/main/java/org/apache/sqoop/client/shell/ShowConnectionFunction.java 4e49288 client/src/main/java/org/apache/sqoop/client/shell/ShowConnectorFunction.java ac2683c client/src/main/java/org/apache/sqoop/client/shell/ShowFrameworkFunction.java 8dcf976 client/src/main/java/org/apache/sqoop/client/shell/ShowJobFunction.java ee8c63d client/src/main/java/org/apache/sqoop/client/shell/ShowServerFunction.java 039e28b client/src/main/java/org/apache/sqoop/client/shell/ShowVersionFunction.java 9e8c607 client/src/main/java/org/apache/sqoop/client/shell/SqoopCommand.java 9ae693e client/src/main/java/org/apache/sqoop/client/shell/SqoopFunction.java 200b3ee client/src/main/java/org/apache/sqoop/client/shell/SqoopShell.java b2d05f4 client/src/main/java/org/apache/sqoop/client/shell/SubmissionCommand.java 39a2b31 client/src/main/java/org/apache/sqoop/client/shell/SubmissionStartFunction.java 74ce905 client/src/main/java/org/apache/sqoop/client/shell/SubmissionStatusFunction.java 4d683c0 client/src/main/java/org/apache/sqoop/client/shell/SubmissionStopFunction.java 97628f7 client/src/main/java/org/apache/sqoop/client/shell/UpdateCommand.java 5bac209 client/src/main/java/org/apache/sqoop/client/shell/UpdateConnectionFunction.java 4e55dba client/src/main/java/org/apache/sqoop/client/shell/UpdateJobFunction.java f7cdf26 client/src/main/resources/client-resource.properties 201efe9 Diff: https://reviews.apache.org/r/8305/diff/ Testing ------- Ran the SQOOP2 client tests and manually ran various client commands to make sure that all commands have their localizable strings and constants properly displayed apart from running all the unit tests. No new tests were added Thanks, Venkat Ranganathan
