[FLINK-33000] SqlGatewayServiceITCase should utilize TestExecutorExtension instead of using a ThreadFactory

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.16.2, 1.18.0, 1.17.1, 1.19.0
• Fix Version/s: 1.16.3, 1.17.2, 1.19.0, 1.18.1
• Component/s: Table SQL / Gateway, Tests
• Labels: pull-request-available starter
• Assignee: jiabao.sun
• Reporter: Matthias Pohl
• Created: 30/Aug/23 13:34
• Updated: 05/Oct/23 04:49
• Resolved: 05/Oct/23 04:42

Description

SqlGatewayServiceITCase uses a ExecutorThreadFactory for its asynchronous operations. Instead, one should use TestExecutorExtension to ensure proper cleanup of threads.

We might also want to remove the AbstractTestBase parent class because that uses JUnit4 whereas SqlGatewayServiceITCase is already based on JUnit5

Issue Links

• links to: GitHub Pull Request #23449
• links to: GitHub Pull Request #23473
• links to: GitHub Pull Request #23474
• links to: GitHub Pull Request #23475