[
https://issues.apache.org/jira/browse/FLINK-14720?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16973142#comment-16973142
]
Gary Yao commented on FLINK-14720:
----------------------------------
Looks correct.
> Bring down ExecutionVertex#deployToSlot access modifier and mark it with
> @VisibleForTesting annotation
> ------------------------------------------------------------------------------------------------------
>
> Key: FLINK-14720
> URL: https://issues.apache.org/jira/browse/FLINK-14720
> Project: Flink
> Issue Type: Improvement
> Reporter: vinoyang
> Priority: Minor
>
> From tracking the call chain of {{ExecutionVertex#deployToSlot}}, it seems
> this method is only been called in the test code. IMO, we'd better bring down
> {{ExecutionVertex#deployToSlot}} access modifier and mark it with
> @VisibleForTesting annotation to reduce the risk of incorrect calls
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
