[
https://issues.apache.org/jira/browse/FLINK-14720?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
vinoyang updated FLINK-14720:
-----------------------------
Summary: Mark ExecutionVertex#deployToSlot with @VisibleForTesting
annotation (was: Bring down ExecutionVertex#deployToSlot access modifier and
mark it with @VisibleForTesting annotation)
> Mark ExecutionVertex#deployToSlot with @VisibleForTesting annotation
> --------------------------------------------------------------------
>
> Key: FLINK-14720
> URL: https://issues.apache.org/jira/browse/FLINK-14720
> Project: Flink
> Issue Type: Improvement
> Reporter: vinoyang
> Assignee: vinoyang
> Priority: Minor
> Labels: pull-request-available
> Time Spent: 10m
> Remaining Estimate: 0h
>
> 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)
