embulk / embulk-base-restclient

Base class library for Embulk plugins to access RESTful services
https://www.embulk.org/
Apache License 2.0
6 stars 7 forks source link

Response.Listener is supposed to be treated on request basis #110

Open shroman opened 5 years ago

shroman commented 5 years ago

With *ResponseEntityReader, a listener is created once. However it's a request-based thing and will fail on a new request when a new request is done (for instance, when retried with Jetty93RetryHelper). Therefore, it should be considered on request basis (ideally, a reader shouldn't own listener).