> On 2012-04-25 01:23:18, Vandana Ayyalasomayajula wrote: > > http://svn.apache.org/repos/asf/incubator/hcatalog/branches/branch-0.4/src/test/org/apache/hcatalog/pig/TestHCatStorer.java, > > line 450 > > <https://reviews.apache.org/r/4861/diff/1/?file=104130#file104130line450> > > > > instead of having another column added ( the 'g' column'), can we have > > the input[2] have the f column null, so that it tests your case. > > > > Like: > > input[i++]= i + "\t" + i * 2.1f +"\t"+ i*1.1d + "\t" + i * 2L > > +"\t"+"lets hcat";
If you see input[i++]= "0\t\t\t\t\t\t"; //Empty values except first column already has empty value for the binary field f. The problem is pig stores a empty byte array when there is a empty value for a bytearray column but null for all other data types. Only way to store null in a bytearray column using pig was to not make it not part of the pig schema. So had to add another column g which was part of hcat schema but not part of pig load schema. - Rohini ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4861/#review7201 ----------------------------------------------------------- On 2012-04-24 22:07:18, Rohini Palaniswamy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4861/ > ----------------------------------------------------------- > > (Updated 2012-04-24 22:07:18) > > > Review request for hcatalog. > > > Summary > ------- > > Fixed the NPE > > > This addresses bug HCATALOG-382. > https://issues.apache.org/jira/browse/HCATALOG-382 > > > Diffs > ----- > > > http://svn.apache.org/repos/asf/incubator/hcatalog/branches/branch-0.4/src/java/org/apache/hcatalog/pig/PigHCatUtil.java > 1329472 > > http://svn.apache.org/repos/asf/incubator/hcatalog/branches/branch-0.4/src/test/org/apache/hcatalog/pig/TestHCatStorer.java > 1329472 > > Diff: https://reviews.apache.org/r/4861/diff > > > Testing > ------- > > Unit test added. > > > Thanks, > > Rohini > >
