...

• A unit test was written to validate the behavior of the memory pool
• A unit test that validates correct behavior of RequestChannel under capacity bounds would need to be written.
• A micro-benchmark for determining the performance of the pool would need to be written
• Stress testing a broker (heavy producer load of varying request sizes) to verify that the memory limit is honored.
• Benchmarking producer and consumer throughput before/after the change to prove that ingress/egress performance remains acceptable.
• Testing of SSL connections (both inter-broker and client-broker) since implementation bugs may affect ssl

Rejected Alternatives

Here are some alternatives that we have discussed (at LinkedIn):

...