reta commented on PR #3235:
URL: https://github.com/apache/cxf/pull/3235#issuecomment-4743179126

   > @reta Damn. One needs to be extremely careful here.
   
   Thanks a lot for the review, @ok2c , really appreciate it
   
   > The trouble is the actual data stream producer is hidden behind that 
InputStream and it basically has no way of pushing data through 
DataStreamChannel or requesting output. That means the channel will remain 
stuck in sleep mode (no output interest).
   
   Agree, I also found no better way but rely on `InputStream::available()` 
(which may return 0), the only good news here -  by design (javadoc at least), 
this method should not block.
   
   > an you represent your data as anything other than InputSteam?
   
   We sadly support dual (buffer / `InputStream`) for a legacy reasons, 
replacing it would require large scale changes, I will look into it, thank you.
   
   


-- 
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]

Reply via email to