Github user zentol commented on the issue: https://github.com/apache/flink/pull/4591 I'm wondering about the benefit here. The existing code guarantees that all buffers would be recycled in case of test failure; don't these changes allow a memory leak? Isn't `recycle()` safe anyway since it is a No-Op if called multiple times without `retain()`?
---