-----------------------------------------------------------
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
|