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()`?---
