----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35122/#review87014 -----------------------------------------------------------
Seems cool. Did you try to play around the binary to make sure nothing obvious popped up? repository/repository-common/src/main/java/org/apache/sqoop/repository/common/CommonRepositoryHandler.java <https://reviews.apache.org/r/35122/#comment139266> Change message to be like the one below? "Looking up link with id: ..." Consistency helps when parsing logs! repository/repository-common/src/main/java/org/apache/sqoop/repository/common/CommonRepositoryHandler.java <https://reviews.apache.org/r/35122/#comment139268> Change message to be like the one below? "Looking up link with name: ..." Consistency helps when parsing logs! repository/repository-common/src/main/java/org/apache/sqoop/repository/common/CommonRepositoryHandler.java <https://reviews.apache.org/r/35122/#comment139269> Change message to be like the one below? "Looking up job with id: ..." Consistency helps when parsing logs! repository/repository-common/src/main/java/org/apache/sqoop/repository/common/CommonRepositoryHandler.java <https://reviews.apache.org/r/35122/#comment139270> Change message to be like the one below? "Looking up job with name: ..." Consistency helps when parsing logs! - Abraham Elmahrek On June 5, 2015, 12:07 p.m., Dian Fu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/35122/ > ----------------------------------------------------------- > > (Updated June 5, 2015, 12:07 p.m.) > > > Review request for Sqoop. > > > Bugs: sqoop-2390 > https://issues.apache.org/jira/browse/sqoop-2390 > > > Repository: sqoop-sqoop2 > > > Description > ------- > > findConnector return null when connector doesn't exist, while findJob and > findLink throw exception when job and link doesn't exist. This is > inconsistent. > > > Diffs > ----- > > common/src/main/java/org/apache/sqoop/error/code/CommonRepositoryError.java > e5fbe2d > > repository/repository-common/src/main/java/org/apache/sqoop/repository/common/CommonRepositoryHandler.java > 774c3b4 > > repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestJobHandling.java > 6f975a6 > > repository/repository-derby/src/test/java/org/apache/sqoop/repository/derby/TestLinkHandling.java > 1ee7996 > > repository/repository-postgresql/src/test/java/org/apache/sqoop/integration/repository/postgresql/TestJobHandling.java > ad601b4 > > repository/repository-postgresql/src/test/java/org/apache/sqoop/integration/repository/postgresql/TestLinkHandling.java > 35736d4 > > Diff: https://reviews.apache.org/r/35122/diff/ > > > Testing > ------- > > > Thanks, > > Dian Fu > >
