Github user ok2c commented on the issue:
https://github.com/apache/httpcomponents-core/pull/91
@rschmitt The proposed interface does look simpler and more aesthetically
pleasing. If we can get it to work there will be no questions asked.
> This is true, but I think it can be addressed simply by decoupling
buffering and processing.
True, but I suppose not every data stream may need buffering. This change
would make it mandatory if I understand it correctly.
> In reactive streaming, we signal exactly how much more data we can
process; we don't explicitly tell the producer to stop, and the producer never
sends us more data than we requested.
This does sound reasonable and this is where we should probably start.
When a particular change is too big and too disruptive what tends to work
reasonably well is changing things one small thing at a time. Let's leave
`AsyncDataConsumer` be for now but get rid of return type in
AsyncDataConsumer#consume as the first step.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
