[jira] [Updated] (FLINK-18355) Simplify tests of SlotPoolImpl

["ASF GitHub Bot (Jira)" <ji...@apache.org>]

     [ 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)