Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Threaded View
Pig, mail # dev - Review Request 19724: PIG-3814: Implement RANK in Tez


Copy link to this message
-
Re: Review Request 19724: PIG-3814: Implement RANK in Tez
Cheolsoo Park 2014-03-27, 18:27

This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19724/#review38772

Ship it!
Thank you for the awesome patch!
http://svn.apache.org/repos/asf/pig/branches/tez/src/org/apache/pig/backend/hadoop/executionengine/tez/TezDagBuilder.java
<https://reviews.apache.org/r/19724/#comment71058>

    This sounds like a good refactoring.
    
    Just curious. Are we going to do the same for POLR too, or is this just for POVO?

http://svn.apache.org/repos/asf/pig/branches/tez/src/org/apache/pig/backend/hadoop/executionengine/tez/operators/POCounterStatsTez.java
<https://reviews.apache.org/r/19724/#comment71051>

    Just a question. Should we move all the Tez physical operators under this package?
- Cheolsoo Park
On March 27, 2014, 5:30 p.m., Rohini Palaniswamy wrote: