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

(Updated Oct. 30, 2014, 9:49 a.m.)


Review request for Sqoop.


Changes
-------

Also did a rebase.


Repository: sqoop-sqoop2


Description
-------

see JIRA and its parent JIRA for details

All the WS stuff, I will do it in the end once the functionality is reviewd.


If someone is wondering why START is changed to submit: there are tons of 
places in the code and in the java docs, we actually mean submit when we say 
START

  DRIVER_0008("Invalid combination of submission and execution engines"),

  DRIVER_0009("Job has been disabled. Cannot submit this job."),

  DRIVER_0010("Link for this job has been disabled. Cannot submit this job."),

  DRIVER_0011("Connector does not support specified direction. Cannot submit 
this job."),

  ;


Diffs (updated)
-----

  client/src/main/java/org/apache/sqoop/client/SqoopClient.java 33a0c3c 
  client/src/main/java/org/apache/sqoop/client/SubmissionCallback.java de7211a 
  client/src/main/java/org/apache/sqoop/client/request/JobResourceRequest.java 
83c08b3 
  
client/src/main/java/org/apache/sqoop/client/request/SqoopResourceRequests.java 
4a56bb7 
  
client/src/main/java/org/apache/sqoop/client/request/SubmissionResourceRequest.java
 5055783 
  common/src/main/java/org/apache/sqoop/json/JobBean.java 082d591 
  common/src/main/java/org/apache/sqoop/json/JobsBean.java PRE-CREATION 
  common/src/main/java/org/apache/sqoop/json/JsonBean.java ba86511 
  common/src/main/java/org/apache/sqoop/json/LinksBean.java 5858a18 
  common/src/main/java/org/apache/sqoop/json/SubmissionBean.java 4b80338 
  common/src/main/java/org/apache/sqoop/json/SubmissionsBean.java PRE-CREATION 
  common/src/main/java/org/apache/sqoop/model/MLink.java 7a9f538 
  common/src/main/java/org/apache/sqoop/model/MSubmission.java 7290df5 
  common/src/test/java/org/apache/sqoop/json/TestJobBean.java 1fc8dbd 
  common/src/test/java/org/apache/sqoop/json/TestLinkBean.java 811cbf0 
  common/src/test/java/org/apache/sqoop/json/TestSubmissionBean.java e4d50bf 
  
connector/connector-generic-jdbc/src/main/java/org/apache/sqoop/connector/jdbc/GenericJdbcPartitioner.java
 2411169 
  
connector/connector-generic-jdbc/src/test/java/org/apache/sqoop/connector/jdbc/TestPartitioner.java
 3ae64f0 
  core/src/main/java/org/apache/sqoop/driver/JobManager.java f6447c6 
  core/src/main/java/org/apache/sqoop/repository/JdbcRepository.java 2aeb07e 
  core/src/main/java/org/apache/sqoop/repository/JdbcRepositoryHandler.java 
ad380d3 
  core/src/main/java/org/apache/sqoop/repository/Repository.java 79742b9 
  
repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbyRepositoryHandler.java
 b996a0b 
  
repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbySchemaInsertUpdateDeleteSelectQuery.java
 c894d06 
  
repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestSubmissionHandling.java
 4c2d062 
  server/src/main/java/org/apache/sqoop/handler/ConnectorRequestHandler.java 
5694ea5 
  server/src/main/java/org/apache/sqoop/handler/JobRequestHandler.java 5547988 
  server/src/main/java/org/apache/sqoop/handler/LinkRequestHandler.java 35a9635 
  server/src/main/java/org/apache/sqoop/handler/SubmissionRequestHandler.java 
8555b0c 
  server/src/main/java/org/apache/sqoop/server/RequestHandler.java 508edd2 
  server/src/main/java/org/apache/sqoop/server/v1/JobServlet.java d295237 
  server/src/main/java/org/apache/sqoop/server/v1/JobsServlet.java PRE-CREATION 
  server/src/main/java/org/apache/sqoop/server/v1/LinkServlet.java 127903a 
  server/src/main/java/org/apache/sqoop/server/v1/SubmissionServlet.java 
5c1d883 
  server/src/main/java/org/apache/sqoop/server/v1/SubmissionsServlet.java 
PRE-CREATION 
  server/src/main/webapp/WEB-INF/web.xml 6ad90d2 
  shell/src/main/java/org/apache/sqoop/shell/AbortCommand.java PRE-CREATION 
  shell/src/main/java/org/apache/sqoop/shell/ShowJobStatusFunction.java 
PRE-CREATION 
  shell/src/main/java/org/apache/sqoop/shell/SqoopShell.java 2e87965 
  shell/src/main/java/org/apache/sqoop/shell/StartCommand.java 7c56980 
  shell/src/main/java/org/apache/sqoop/shell/StartJobFunction.java 4363f05 
  shell/src/main/java/org/apache/sqoop/shell/StatusCommand.java 3447a87 
  shell/src/main/java/org/apache/sqoop/shell/StatusJobFunction.java fb83af3 
  shell/src/main/java/org/apache/sqoop/shell/StopCommand.java 50b2e81 
  shell/src/main/java/org/apache/sqoop/shell/StopJobFunction.java 790c522 
  shell/src/main/java/org/apache/sqoop/shell/SubmitCommand.java PRE-CREATION 
  shell/src/main/java/org/apache/sqoop/shell/SubmitJobFunction.java 
PRE-CREATION 
  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/core/Constants.java 44d5920 
  shell/src/main/resources/shell-resource.properties 0e63c50 
  test/src/main/java/org/apache/sqoop/test/testcases/ConnectorTestCase.java 
af31769 
  
test/src/test/java/org/apache/sqoop/integration/connector/jdbc/generic/FromRDBMSToHDFSTest.java
 36f7443 
  tools/src/main/java/org/apache/sqoop/tools/tool/RepositoryDumpTool.java 
c219e68 
  tools/src/main/java/org/apache/sqoop/tools/tool/RepositoryLoadTool.java 
64b08fc 

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


Testing
-------

yes


Thanks,

Veena Basavaraj

Reply via email to