sqoop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Veena Basavaraj" <vbasava...@cloudera.com>
Subject Re: Review Request 27236: SQOOP-1513: Link rest api changes
Date Tue, 28 Oct 2014 01:41:43 GMT 

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

(Updated Oct. 27, 2014, 6:41 p.m.)


Review request for Sqoop.


Repository: sqoop-sqoop2


Description (updated)
-------

see JIRA for details.

Note :I just realized that it might really be more work in HUE to have th parent/child in
URLS,

so v1/link ? connector-id in the post data for creating a link rather than v1/connector/{cId}/link


Diffs (updated)
-----

  client/src/main/java/org/apache/sqoop/client/request/LinkResourceRequest.java 64e5cb1 
  common/src/main/java/org/apache/sqoop/json/LinkBean.java 2762500 
  common/src/main/java/org/apache/sqoop/json/LinksBean.java PRE-CREATION 
  common/src/test/java/org/apache/sqoop/json/TestLinkBean.java 526ec52 
  core/src/main/java/org/apache/sqoop/repository/JdbcRepository.java 254ba9e 
  core/src/main/java/org/apache/sqoop/repository/JdbcRepositoryHandler.java 7d78826 
  core/src/main/java/org/apache/sqoop/repository/Repository.java fcee48b 
  repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbyRepositoryHandler.java
7f19c28 
  repository/repository-derby/src/main/java/org/apache/sqoop/repository/derby/DerbySchemaInsertUpdateDeleteSelectQuery.java
02b11fc 
  repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestLinkHandling.java
37343d3 
  server/src/main/java/org/apache/sqoop/handler/LinkRequestHandler.java 823426a 
  server/src/main/java/org/apache/sqoop/server/RequestContext.java b1a74fc 
  server/src/main/java/org/apache/sqoop/server/common/ServerError.java 02d9174 
  server/src/main/java/org/apache/sqoop/server/v1/ConnectorServlet.java 781abc3 
  server/src/main/java/org/apache/sqoop/server/v1/DriverServlet.java 43454fd 
  server/src/main/java/org/apache/sqoop/server/v1/LinkServlet.java 20e5009 
  server/src/main/java/org/apache/sqoop/server/v1/LinksServlet.java PRE-CREATION 
  server/src/main/webapp/WEB-INF/web.xml a6f7b0d 

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


Testing
-------

yes


Thanks,

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