[FLINK-8172] Remove unnecessary synchronisation in RecordSerializer

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.3.2, 1.4.0
• Fix Version/s: 1.5.0
• Component/s: Runtime / Network
• Assignee: Piotr Nowojski
• Reporter: Piotr Nowojski
• Created: 29/Nov/17 17:11
• Updated: 12/Feb/18 11:59
• Resolved: 12/Feb/18 11:59

Description

While writing the records, RecordSerializer is the only owner of the `Buffer` into which data are written. Yet we are synchronisation twice per record while accessing MemorySegment. Removing this synchronisation speeds up the Network throughput in point to point benchmark by a factor of two (from ~12500records/ms up to 23000 records/ms).

Issue Links

• is superceded by: FLINK-8178 Introduce not threadsafe write only BufferBuilder
• links to: GitHub Pull Request #5104