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

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

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

Shang Yuanchun (Jira)
Zhu Zhu created FLINK-18355:
-------------------------------

             Summary: 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
             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)