chibenwa commented on PR #2137: URL: https://github.com/apache/james-project/pull/2137#issuecomment-2010663386
https://github.com/apache/james-project/pull/2138 is NOT a solution. ChunkStream is necessarily blocking on slow data supplier cf https://github.com/netty/netty/issues/2984 I will be closing this as such. So far I see 2 possible solution: - Switch to a distinct EventExecutor for some parts of the Netty pipeline. Early tries shows at least fine tunning would be needed. Benefit: limited refactoring needs. Limits: realies on blocking abstractions, costs of switching to a distinct thread. - Carry other the `Flux<ByteBuffer>` and write it in a backpressure aware fashion. Fully compatible with reactive code. Requires refactoring of BlobStore, Mailbox and IMAP. More importantly the AES blob store would be non complying as it relies on `InputStream` absctraction... -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
