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

Daniel Dai commented on PIG-3457:
---------------------------------

Thanks [~cheolsoo], you explained the patch very well.

[~rohini], for your comment:
bq. 1) Can we name it PigStatsBase and get rid of Util?
I feel that is more confusing, it gives the impression PigStats extends 
PigStatsBase. However, it is a separate family: PigStatsUtilBase -> 
MRPigStatsUtil -> PigStatsUtil

bq. 2) Can we mark both the new classes as Evolving?
I think we will have a follow up for PIG-3419 to remark all new interfaces. But 
if you want me to mark the affected classes in this patch, I can also do that.
                
> 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
>
>         Attachments: PIG-3457-1.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