[ 
https://issues.apache.org/jira/browse/PIG-3457?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13780361#comment-13780361
 ] 

Cheolsoo Park commented on PIG-3457:
------------------------------------

Thank you Daniel for taking care of this. Two questions:
1) Can we commit this only to branch-0.12? We don't need to carry this on in 
0.13, do we?
2) I don't seem to understand why it's necessary for JobStats class to have the 
private MRJobStats member while it extends MRJobStats. Can you help me to 
understand this?
{code}
 public class JobStats extends MRJobStats {
+    MRJobStats jobStats;
     public JobStats(String name, JobGraph plan) {
         super(name, plan);
     }
+    public JobStats(MRJobStats mrJobStats) {
+        super(mrJobStats.getName(), (JobGraph)mrJobStats.getPlan());
+        this.jobStats = mrJobStats;
+    }
+    
+    @Override
+    public JobState getState() { return jobStats.getState(); }
...
{code}
                
> Provide backward compatibility for PigStatsUtil and JobStats
> ------------------------------------------------------------
>
>                 Key: PIG-3457
>                 URL: https://issues.apache.org/jira/browse/PIG-3457
>             Project: Pig
>          Issue Type: Bug
>          Components: impl
>            Reporter: Daniel Dai
>            Assignee: Daniel Dai
>             Fix For: 0.12.0
>
>         Attachments: PIG-3457-1.patch, PIG-3457-2.patch, PIG-3457-3.patch, 
> PIG-3457-4.patch, PIG-3457-5.patch
>
>
> PIG-3419 restructured PigStatsUtil which break downstream projects such as 
> Oozie. Oozie uses PigStatsUtil.{HDFS_BYTES_WRITTEN, MAP_INPUT_RECORDS, 
> MAP_OUTPUT_RECORDS, REDUCE_INPUT_RECORDS, REDUCE_OUTPUT_RECORDS}. We need to 
> provide a backward compatible way.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to