[
https://issues.apache.org/jira/browse/FLINK-18355?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
ASF GitHub Bot updated FLINK-18355:
-----------------------------------
Labels: pull-request-available (was: )
> Simplify tests of SlotPoolImpl
> ------------------------------
>
> Key: FLINK-18355
> URL: https://issues.apache.org/jira/browse/FLINK-18355
> Project: Flink
> Issue Type: Sub-task
> Components: Runtime / Coordination, Tests
> Reporter: Zhu Zhu
> Assignee: Zhilong Hong
> Priority: Major
> Labels: pull-request-available
> Fix For: 1.12.0
>
>
> Tests of SlotPoolImpl, including SlotPoolImplTest, SlotPoolInteractionsTest
> and SlotPoolSlotSharingTest, are somehow unnecessarily complicated in the
> code involvement. E.g. SchedulerImp built on top of SlotPoolImpl is used to
> allocate slots from SlotPoolImpl, which can be simplified by directly invoke
> slot allocation on SlotPoolImpl.
> Besides that, there are quite some duplications between tests classes of
> SlotPoolImpl, this further includes SlotPoolPendingRequestFailureTest,
> SlotPoolRequestCompletionTest and SlotPoolBatchSlotRequestTest.
> It can ease future development and maintenance a lot if we clean up these
> tests by
> 1. introduce a comment test base for fields and methods reuse
> 2. remove the usages of SchedulerImp for slotpool testing
> 3. other possible simplifications
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
