----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8305/ -----------------------------------------------------------
(Updated Dec. 8, 2012, 5:58 p.m.) Review request for Sqoop and Jarek Cecho. Changes ------- Hi Jarcec Thanks for reviewing. I have updated the client resource to add the new version string and also updated ShowConnector.java. I have tested to make sure that the Version string is printed in show connector option 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 4df1c71 client/src/main/java/org/apache/sqoop/client/shell/CreateJobFunction.java 3aa6c4f 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 3116cd9 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
