lgoldstein commented on a change in pull request #181:
URL: https://github.com/apache/mina-sshd/pull/181#discussion_r584085459
##########
File path:
sshd-core/src/main/java/org/apache/sshd/server/forward/TcpipServerChannel.java
##########
@@ -215,10 +215,12 @@ protected OpenFuture doInit(Buffer buffer) {
}
if (streaming == Streaming.Async) {
+ int channelId = getId();
out = new BufferedIoOutputStream(
Review comment:
As far as I can tell this code does not affect the issues you mentioned.
IMO it is aligned with the purpose of PR#166 of keeping the in-flight data size
under control.
>> How does this interact with this new mechanism?
I think it "plugs a hole" so to speak since the low-level mechanism is not
aware of the concerns raised in the `BufferIoOutputStream` pending write
requests mechanism.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
