----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25930/#review54409 -----------------------------------------------------------
Ship it! ivy.xml <https://reviews.apache.org/r/25930/#comment94543> This change has been already committed, so the patch won't apply with it. - Jarek Cecho On Sept. 23, 2014, 7:40 a.m., Venkat Ranganathan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/25930/ > ----------------------------------------------------------- > > (Updated Sept. 23, 2014, 7:40 a.m.) > > > Review request for Sqoop. > > > Bugs: SQOOP-1539 > https://issues.apache.org/jira/browse/SQOOP-1539 > > > Repository: sqoop-trunk > > > Description > ------- > > Add additional validation of partition keys and values. Empty partition keys > and values will be checked for and reported. > > > Diffs > ----- > > ivy.xml 2759755 > src/java/org/apache/sqoop/mapreduce/hcat/SqoopHCatUtilities.java b69b83e > src/java/org/apache/sqoop/tool/BaseSqoopTool.java 3104454 > src/test/org/apache/sqoop/hcat/TestHCatalogBasic.java 388cce5 > > Diff: https://reviews.apache.org/r/25930/diff/ > > > Testing > ------- > > Added a unit test to test the scenario. All tests pass > > > Thanks, > > Venkat Ranganathan > >
