[FLINK-21517] Test harnesses are bypassing serialization stack for events

• Type: Bug
• Status: Closed
• Priority: Critical
• Resolution: Fixed
• Affects Version/s: 1.12.1, 1.13.0
• Fix Version/s: 1.13.0
• Component/s: Runtime / Network, Tests
• Labels: pull-request-available
• Assignee: Unassigned
• Reporter: Piotr Nowojski
• Created: 26/Feb/21 12:54
• Updated: 27/Feb/21 06:21
• Resolved: 27/Feb/21 06:21

Description

Since FLINK-19297 (https://github.com/apache/flink/pull/13447/files#diff-e5c3ecec28e8d4c2f7f62bc8a4c9ed88c30141d44a570059849a3b2120e4d2b50) we accidentally removed a test coverage for event (de)serialization from a lot of the unit tests, that were/are using test harnesses. For example because of that I almost broke 1.12.2 release, since `stop-with-savepoint --drain` was only tested using test harnesses (didn't have an ITCases and/or end to end test).

Issue Links

• links to: GitHub Pull Request #15036