jetty.project icon indicating copy to clipboard operation
jetty.project copied to clipboard

Review LeakDetector & LeakTrackingConnectionPool

Open sbordet opened this issue 5 years ago • 2 comments

Jetty version 9.4.x

Description Review LeakDetector to see if it's modern enough.

Also, try to make LeakTrackingConnectionPool use delegation rather than inheritance, following the model of LeakTrackingByteBufferPool.

See also #2814 for a description of the problems that LeakTrackingConnectionPool has due to the fact that uses delegation.

Benchmark the cost of the leak detector to see how expensive it is, to suggest whether it's fine to use it in "production" code or not.

sbordet avatar Jun 08 '20 16:06 sbordet

This issue has been automatically marked as stale because it has been a full year without activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Jun 09 '21 02:06 stale[bot]

This issue has been automatically marked as stale because it has been a full year without activity. It will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Jun 11 '22 00:06 github-actions[bot]

This issue has been automatically marked as stale because it has been a full year without activity. It will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Jun 14 '23 00:06 github-actions[bot]