Re: Review Request: Refactor physical operators to remove methods parameters that are always null

2013-05-17 Thread Julien Le Dem
> On May 17, 2013, 3:12 p.m., Cheolsoo Park wrote: > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/POCast.java, > > lines 394-395 > > > > > > This isn't what you introduced, but

Re: Review Request: Refactor physical operators to remove methods parameters that are always null

2013-05-17 Thread Cheolsoo Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11203/#review20674 --- Ship it! Looks good to me. I only have minor comments as below. Do

Review Request: Refactor physical operators to remove methods parameters that are always null

2013-05-16 Thread Julien Le Dem
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11203/ --- Review request for pig, Daniel Dai, Dmitriy Ryaboy, Cheolsoo Park, and Bill Grah