Skip to content

ByteBuf leak fixes#1876

Open
rozza wants to merge 2 commits intomongodb:ByteBuffrom
rozza:JAVA-6081
Open

ByteBuf leak fixes#1876
rozza wants to merge 2 commits intomongodb:ByteBuffrom
rozza:JAVA-6081

Conversation

@rozza
Copy link
Member

@rozza rozza commented Feb 2, 2026

  • Ensure Default Server Monitor calls close on resources before interrupt
  • Update ByteBufferBsonOutput documentation
  • Improve ReplyHeader testing and ensure resources are closed
  • Improve ServerSessionPool testing
  • Ensure reactive client session closing is idempotent
  • Added System.gc to unified test cleanup. Should cause more gc when testing.

JAVA-6081

- Ensure Default Server Monitor calls close on resources before interrupt
- Update ByteBufferBsonOutput documentation
- Improve ReplyHeader testing and ensure resources are closed
- Improve ServerSessionPool testing
- Ensure reactive client session closing is idempotent
- Added System.gc to unified test cleanup. Should cause more gc when testing.

JAVA-6081
@rozza rozza requested a review from a team as a code owner February 2, 2026 15:11
@rozza rozza requested a review from katcharov February 2, 2026 15:11
@rozza
Copy link
Member Author

rozza commented Feb 2, 2026

DefaultServerMonitor

  • Resource cleanup before thread interrupt
  • Prevents leaks during shutdown scenarios

Other changes

  • Test migrations
  • Improved documentation in ByteBufBsonOutput
  • Improved resource handling in BsonDocument & legacy
  • Updated reactive stream Fixture to increase use of Netty when transport setting configured
  • Updated unified tests to increase the gc churn

@rozza rozza requested a review from stIncMale February 2, 2026 15:14
@strogiyotec strogiyotec mentioned this pull request Feb 9, 2026
@katcharov katcharov requested review from nhachicha and removed request for katcharov and stIncMale February 19, 2026 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments