You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sqoop.apache.org by richard zhou <ri...@intel.com> on 2015/02/27 07:15:22 UTC

Review Request 31527: SQOOP-2139: Change create role API from PUT to POST

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

Review request for Sqoop and Abraham Elmahrek.


Repository: sqoop-sqoop2


Description
-------

Change create role API from PUT to POST


Diffs
-----

  client/src/main/java/org/apache/sqoop/client/request/AuthorizationResourceRequest.java cdd8e7305432ffe6360042c4e1638f50e3bcb42a 
  server/src/main/java/org/apache/sqoop/handler/AuthorizationRequestHandler.java 361d14b409e019f066a3db2052d72209e59c097e 
  server/src/main/java/org/apache/sqoop/server/AuthorizationServlet.java 65c434d200305dcdb7e3e9bd4cf3d153f5f7a93c 

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


Testing
-------


Thanks,

richard zhou


Re: Review Request 31527: SQOOP-2139: Change create role API from PUT to POST

Posted by Abraham Elmahrek <ab...@cloudera.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31527/#review74595
-----------------------------------------------------------

Ship it!


Ship It!

- Abraham Elmahrek


On Feb. 27, 2015, 6:15 a.m., richard zhou wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/31527/
> -----------------------------------------------------------
> 
> (Updated Feb. 27, 2015, 6:15 a.m.)
> 
> 
> Review request for Sqoop and Abraham Elmahrek.
> 
> 
> Repository: sqoop-sqoop2
> 
> 
> Description
> -------
> 
> Change create role API from PUT to POST
> 
> 
> Diffs
> -----
> 
>   client/src/main/java/org/apache/sqoop/client/request/AuthorizationResourceRequest.java cdd8e7305432ffe6360042c4e1638f50e3bcb42a 
>   server/src/main/java/org/apache/sqoop/handler/AuthorizationRequestHandler.java 361d14b409e019f066a3db2052d72209e59c097e 
>   server/src/main/java/org/apache/sqoop/server/AuthorizationServlet.java 65c434d200305dcdb7e3e9bd4cf3d153f5f7a93c 
> 
> Diff: https://reviews.apache.org/r/31527/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> richard zhou
> 
>