> On Sep 5, 2018, at 9:36 AM, Bryan Call <bc...@apache.org> wrote:
>
> How about warning periodically or having a configuration option to disable
> the error message?
I think rate limiting warnings is a good idea. +1. Heck, I might even suggest
that Warning() ought to have that built in :-).
I looked at the patch in #2 above, and it seems that could lose useful
information? Presumably you might want to know that something has gone wrong
(the missing reader) ? It’s not a totally unreadable patch, I’m just concerned
that you are missing out on information that is useful.
Cheers,
— Leif
