[
https://issues.apache.org/jira/browse/TEZ-1089?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14004689#comment-14004689
]
Chen He edited comment on TEZ-1089 at 5/21/14 2:28 PM:
-------------------------------------------------------
Thank you, [~sseth], patch updated.
1) add the missing unit test file;
2) change the assertion in unit test file;
3) change the variable name in tez-api/src/main/proto/Events.proto file
4) change method name in ProtoConverters
was (Author: airbots):
Thank you, [~sseth], patch updated.
1) add the missing unit test file;
2) change the assertion in unit test file;
3) change the variable name in tez-api/src/main/proto/Events.proto file
> CompositeDataMovementEvent srcIndexEnd is not an index
> ------------------------------------------------------
>
> Key: TEZ-1089
> URL: https://issues.apache.org/jira/browse/TEZ-1089
> Project: Apache Tez
> Issue Type: Bug
> Reporter: Siddharth Seth
> Assignee: Chen He
> Labels: newbie
> Attachments: TEZ-1089-v2.patch, TEZ-1089.patch
>
>
> CompositeDataMovemntEvent accepts indices for start / end. End happens to be
> a count instead of an index - which can get confusing, especially given the
> name (srcIndexEnd).
--
This message was sent by Atlassian JIRA
(v6.2#6252)
