[FLINK-16189] Remove testing logic from FlinkDistribution

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.11.0
• Component/s: Test Infrastructure
• Labels: pull-request-available
• Assignee: Chesnay Schepler
• Reporter: Chesnay Schepler
• Created: 20/Feb/20 14:15
• Updated: 05/Mar/20 15:12
• Resolved: 05/Mar/20 15:12

Description

Currently, the FlinkDistribution is responsible for locating and copying the distribution for usage in tests, and contains hooks for backing up log files.
This makes the class a bit inflexible to use, as for example a FlinkResource could not create 2 distributions in separate places.

I suggest to move these responsibilities into the FlinkResource implementations, and "demote" the distribution to a simple wrapper, only providing programmatic access for mutating/interacting with the distribution.

Issue Links

• links to: GitHub Pull Request #11285