----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17958/#review34223 -----------------------------------------------------------
src/org/apache/pig/backend/hadoop/executionengine/tez/TezDagBuilder.java <https://reviews.apache.org/r/17958/#comment64255> Doesn't this take care of the memory settings for you? - Rohini Palaniswamy On Feb. 11, 2014, 7:42 p.m., Mark Wagner wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/17958/ > ----------------------------------------------------------- > > (Updated Feb. 11, 2014, 7:42 p.m.) > > > Review request for pig, Alex Bain, Cheolsoo Park, Daniel Dai, and Rohini > Palaniswamy. > > > Bugs: PIG-3768 > https://issues.apache.org/jira/browse/PIG-3768 > > > Repository: pig-git > > > Description > ------- > > Instead of creating empty configurations, initialize configurations with the > properties provided by the user. > > > Diffs > ----- > > src/org/apache/pig/backend/hadoop/executionengine/tez/TezDagBuilder.java > b3e74d9 > > Diff: https://reviews.apache.org/r/17958/diff/ > > > Testing > ------- > > Ran test-commit, and ran a script that wasn't respecting memory setting > before, which succeeded after the patch. > > > Thanks, > > Mark Wagner > >
