sqoop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bilung Lee" <bleeapa...@gmail.com>
Subject Re: Review Request: SQOOP-716 Create server notification REST callback to get information if job has update
Date Mon, 26 Nov 2012 17:00:52 GMT

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

Ship it!


Ship It!

- Bilung Lee


On Nov. 25, 2012, 4:55 a.m., Jarek Cecho wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/8211/
> -----------------------------------------------------------
> 
> (Updated Nov. 25, 2012, 4:55 a.m.)
> 
> 
> Review request for Sqoop.
> 
> 
> Description
> -------
> 
> I've added support for notification URL (in sense of mapreduce framework).
> 
> I had issues to get to Sqoop server URL from within the server itself. I've decided to
put the detection code to the submission submit REST end point for now, but we should move
it to more appropriate place in future.
> 
> 
> This addresses bug SQOOP-716.
>     https://issues.apache.org/jira/browse/SQOOP-716
> 
> 
> Diffs
> -----
> 
>   core/src/main/java/org/apache/sqoop/framework/FrameworkManager.java c9c9cf4deb59994da8ebf7937e5f17521fbd8102

>   core/src/main/java/org/apache/sqoop/framework/SubmissionRequest.java 8392a10de2a05b8dc952a384cc193eb2fadc143b

>   server/src/main/java/org/apache/sqoop/handler/SubmissionRequestHandler.java e9e65511c6856728e06516d2e832831d65204453

>   server/src/main/java/org/apache/sqoop/server/ServerInitializer.java ae0735ba112a794dfcec10ea8896c78650b406d6

>   submission/mapreduce/src/main/java/org/apache/sqoop/submission/mapreduce/MapreduceSubmissionEngine.java
68f21fd77e84c5c567058e562d80330c347219c8 
> 
> Diff: https://reviews.apache.org/r/8211/diff/
> 
> 
> Testing
> -------
> 
> Tested on real cluster.
> 
> 
> Thanks,
> 
> Jarek Cecho
> 
>


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