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

Kuhu Shukla edited comment on TEZ-3726 at 5/16/17 9:18 PM:
-----------------------------------------------------------

[~jlowe], patch with additional changes from TEZ-3729 just to make the review 
easier since the diff is not that big. Appreciate any comments/review on this. 
Thanks a lot!


was (Author: kshukla):
[~jlowe], path with additional changes from TEZ-3729 just to make the review 
easier since the diff is not that bug. Appreciate any comments/review on this. 
Thanks a lot!

> DeletionTracker's reflections instantiation should be simpler and generic.
> --------------------------------------------------------------------------
>
>                 Key: TEZ-3726
>                 URL: https://issues.apache.org/jira/browse/TEZ-3726
>             Project: Apache Tez
>          Issue Type: Sub-task
>            Reporter: Kuhu Shukla
>            Assignee: Kuhu Shukla
>         Attachments: TEZ-3726.001.patch, TEZ-3726.002.patch, 
> TEZ-3726.003.patch
>
>
> This JIRA will address the following feedback from [~jlowe].
> {code}
> The reflection instantiation is invoking a constructor signature that isn't 
> in the DeletionTracker abstract class? Isn't that too much knowledge about 
> the actual class being created?
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to