----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4623/#review6762 -----------------------------------------------------------
/trunk/src/java/org/apache/hcatalog/pig/HCatLoader.java <https://reviews.apache.org/r/4623/#comment14913> Would suggest to move this up to where the map is declared. Also comment WRT uniqueness of keys into that map might be useful. - Thomas On 2012-04-06 21:08:16, Vandana Ayyalasomayajula wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4623/ > ----------------------------------------------------------- > > (Updated 2012-04-06 21:08:16) > > > Review request for hcatalog, Francis Liu, Thomas, and Rohini Palaniswamy. > > > Summary > ------- > > HCatStorer.setStoreLocation() is called more than once by pig. Which ends up > calling HCatOutputFormat.setOutput multiple times. HCatOutputFormat.setOutput > should be called only once, the values that it changed must be tracked and > reapplied to job.getconfiguration() the succeeding times it is called. > HCatLoader may have the same issue and should be investigated as well. > > > This addresses bug hcatalog-314. > https://issues.apache.org/jira/browse/hcatalog-314 > > > Diffs > ----- > > /trunk/src/java/org/apache/hcatalog/common/HCatConstants.java 1309184 > /trunk/src/java/org/apache/hcatalog/pig/HCatLoader.java 1309184 > /trunk/src/java/org/apache/hcatalog/pig/HCatStorer.java 1309184 > > Diff: https://reviews.apache.org/r/4623/diff > > > Testing > ------- > > Trunk build + tests pass. > > > Thanks, > > Vandana > >
