----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7268/#review11990 -----------------------------------------------------------
Ship it! Ship It! - Vandana Ayyalasomayajula On Sept. 25, 2012, 8:27 p.m., Travis Crawford wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/7268/ > ----------------------------------------------------------- > > (Updated Sept. 25, 2012, 8:27 p.m.) > > > Review request for hcatalog. > > > Description > ------- > > Update HCatContext to use the given Configuration, rather than using a new > Configuration. When given the jobconf configuration, the MR framework takes > care of passing it to the backend. Currently, the HCatContext configuration > is not passed to the backend. > > > This addresses bug HCATALOG-497. > https://issues.apache.org/jira/browse/HCATALOG-497 > > > Diffs > ----- > > hcatalog-pig-adapter/src/main/java/org/apache/hcatalog/pig/HCatLoader.java > 2b001f8 > hcatalog-pig-adapter/src/main/java/org/apache/hcatalog/pig/HCatStorer.java > 65e799a > src/java/org/apache/hcatalog/common/HCatContext.java f19489c > src/java/org/apache/hcatalog/data/HCatRecordSerDe.java ca204c3 > src/java/org/apache/hcatalog/data/schema/HCatSchemaUtils.java 38a5fa9 > > Diff: https://reviews.apache.org/r/7268/diff/ > > > Testing > ------- > > Manually tested on a fully distributed cluster, reading a table that contains > a smallint field. Trunk fails with the exception posted in Jira; with this > patch the job successfully completes. > > > Thanks, > > Travis Crawford > >
