Bill commented on a change in pull request #7449:
URL: https://github.com/apache/geode/pull/7449#discussion_r839882332
##########
File path:
geode-core/src/main/java/org/apache/geode/internal/tcp/Connection.java
##########
@@ -2527,7 +2527,7 @@ private void writeAsync(SocketChannel channel, ByteBuffer
buffer, boolean forceA
long queueTimeoutTarget = now + asyncQueueTimeout;
channel.configureBlocking(false);
try {
- try (final ByteBufferSharing outputSharing = ioFilter.wrap(buffer)) {
+ try (final ByteBufferSharing outputSharing = ioFilter.wrap(buffer,
channel)) {
Review comment:
Thanks for pointing this out @kirklund. I'd like to do a minimal touch
to product code in this PR so it is easily back-ported. I will make a note to
clean up nested trys in Connection in a future PR.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
