> On Sept. 24, 2014, 12:16 p.m., Jarek Cecho wrote: > > ivy.xml, lines 193-195 > > <https://reviews.apache.org/r/25930/diff/1/?file=702643#file702643line193> > > > > This change has been already committed, so the patch won't apply with > > it.
Thanks for the catch. - Venkat ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25930/#review54409 ----------------------------------------------------------- On Sept. 24, 2014, 9:39 p.m., Venkat Ranganathan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/25930/ > ----------------------------------------------------------- > > (Updated Sept. 24, 2014, 9:39 p.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 > ----- > > 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 > >
