----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24873/#review51060 -----------------------------------------------------------
Ship it! Ship It! - Jarek Cecho On Aug. 19, 2014, 11:53 p.m., Abraham Elmahrek wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/24873/ > ----------------------------------------------------------- > > (Updated Aug. 19, 2014, 11:53 p.m.) > > > Review request for Sqoop. > > > Bugs: SQOOP-1451 > https://issues.apache.org/jira/browse/SQOOP-1451 > > > Repository: sqoop-SQOOP-1367 > > > Description > ------- > > commit 980555e529bebcd874ce331d511bbc4af94fa8b2 > Author: Abraham Elmahrek <[email protected]> > Date: Tue Aug 19 16:00:51 2014 -0700 > > SQOOP-1451: Sqoop2: From/To: Add API for supported directions > > :100644 100644 d17ebef... 8782209... M > core/src/main/java/org/apache/sqoop/connector/ConnectorHandler.java > :100644 100644 f7a6b67... 8ecb197... M > core/src/main/java/org/apache/sqoop/framework/FrameworkError.java > :100644 100644 48360ea... 5571928... M > core/src/main/java/org/apache/sqoop/framework/JobManager.java > :100644 100644 6f2df93... fba5b1c... M > server/src/main/java/org/apache/sqoop/handler/JobRequestHandler.java > :100644 100644 2112bf7... 2774e8b... M > spi/src/main/java/org/apache/sqoop/connector/spi/SqoopConnector.java > > > Diffs > ----- > > core/src/main/java/org/apache/sqoop/connector/ConnectorHandler.java d17ebef > core/src/main/java/org/apache/sqoop/framework/FrameworkError.java f7a6b67 > core/src/main/java/org/apache/sqoop/framework/JobManager.java 48360ea > server/src/main/java/org/apache/sqoop/handler/JobRequestHandler.java > 6f2df93 > spi/src/main/java/org/apache/sqoop/connector/spi/SqoopConnector.java > 2112bf7 > > Diff: https://reviews.apache.org/r/24873/diff/ > > > Testing > ------- > > Tested with fake connector. Seems to work! TODO: Integration test. > > > Thanks, > > Abraham Elmahrek > >
