----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31166/#review73119 -----------------------------------------------------------
Ship it! Ship It! - Abraham Elmahrek On Feb. 19, 2015, 9:52 a.m., Szehon Ho wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/31166/ > ----------------------------------------------------------- > > (Updated Feb. 19, 2015, 9:52 a.m.) > > > Review request for Sqoop. > > > Bugs: SQOOP-2117 > https://issues.apache.org/jira/browse/SQOOP-2117 > > > Repository: sqoop-sqoop2 > > > Description > ------- > > Currently it shows id, which is not helpful unless user shows the connectors > as well. > > This patch during 'show link' make another call on the client side to get > connector info, which more often than not should be cached (such as when the > link is created). > > > Diffs > ----- > > shell/src/main/java/org/apache/sqoop/shell/ShowLinkFunction.java e92f0b4 > shell/src/main/java/org/apache/sqoop/shell/core/Constants.java 46f2b7f > shell/src/main/resources/shell-resource.properties 9c88235 > > Diff: https://reviews.apache.org/r/31166/diff/ > > > Testing > ------- > > Manually tested > > > Thanks, > > Szehon Ho > >
