You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@tez.apache.org by "Shashanka Balakuntala Srinivasa (Jira)" <ji...@apache.org> on 2021/02/17 15:09:00 UTC

[jira] [Commented] (TEZ-138) Merge DAG/DAGImpl, Vertex/VertexImpl, Task/TaskImpl, TaskAttempt/TaskAttemptImpl

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

Shashanka Balakuntala Srinivasa commented on TEZ-138:
-----------------------------------------------------

Hi a small reminder for a review.

> Merge DAG/DAGImpl, Vertex/VertexImpl, Task/TaskImpl, TaskAttempt/TaskAttemptImpl
> --------------------------------------------------------------------------------
>
>                 Key: TEZ-138
>                 URL: https://issues.apache.org/jira/browse/TEZ-138
>             Project: Apache Tez
>          Issue Type: Improvement
>            Reporter: Bikas Saha
>            Priority: Major
>         Attachments: TEZ-138.patch
>
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> We are not going to have another impl of these interfaces. What are they buying us other than overhead of overriding every new public method.
> For tests, every class and every methods can be overridden and mocked.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)