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

Robert Metzger commented on FLINK-21347:
----------------------------------------

I'm not sure if having an interface really makes things much simpler. The 
interface would have ~35 methods + ~15 methods in AccessExecutionGraph. That 
doesn't necessarily sound like a simple interface for testing.
We need to carefully review which methods to expose.

> Extract interface out of ExecutionGraph for better testability
> --------------------------------------------------------------
>
>                 Key: FLINK-21347
>                 URL: https://issues.apache.org/jira/browse/FLINK-21347
>             Project: Flink
>          Issue Type: Task
>          Components: Runtime / Coordination
>            Reporter: Robert Metzger
>            Priority: Major
>
> This is a follow up to this comment: 
> https://github.com/apache/flink/pull/14879#discussion_r573613450
> The ExecutionGraph class is currently not very handy for tests, as it has a 
> lot of dependencies. Extracting an interface for the ExecutionGraph will make 
> testing and future changes easier.



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

Reply via email to