Rename to `bufferBuilder` (marking type in variable name is not the best practice).
Have you seen throughput improvements by introducing this local variable? If not, maybe revert the change? FYI: When I was writing this code, I didn't see any performance improvement (and I was testing this exact change). Removing one extra CPU cache read (second `bufferBuilders[targetChannel]` access will either be optimised out or it will be a read from CPU caches/registries) usually hardly matters compared to taking locks :( [ Full content available at: https://github.com/apache/flink/pull/6417 ] This message was relayed via gitbox.apache.org for [email protected]
