sqoop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From rj.vasanthku...@gmail.com
Subject Review Request: Client shell variable substitution for recent commands
Date Sat, 05 Jan 2013 21:51:32 GMT

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

Review request for Sqoop.


Description
-------

execute() method is replaced with executeCommand() method in all Command classes. executed()
method overrided in SqoopCommand which resolves the variables and then executes the command.


This addresses bug SQOOP-811.
    https://issues.apache.org/jira/browse/SQOOP-811


Diffs
-----

  client/src/main/java/org/apache/sqoop/client/shell/CloneCommand.java 827d168 
  client/src/main/java/org/apache/sqoop/client/shell/CreateCommand.java ad4d8fd 
  client/src/main/java/org/apache/sqoop/client/shell/DeleteCommand.java 5ab6495 
  client/src/main/java/org/apache/sqoop/client/shell/SetCommand.java fb6e8c8 
  client/src/main/java/org/apache/sqoop/client/shell/ShowCommand.java efbb8f2 
  client/src/main/java/org/apache/sqoop/client/shell/SqoopCommand.java d818353 
  client/src/main/java/org/apache/sqoop/client/shell/SubmissionCommand.java 2e1fb03 
  client/src/main/java/org/apache/sqoop/client/shell/UpdateCommand.java d73b40e 

Diff: https://reviews.apache.org/r/8842/diff/


Testing
-------


Thanks,

vasanthkumar


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message