Buffer options retry_forever and secondary shouldn't be exclusive

[yann-soubeyrand]

Hi,

In my opinion, buffer options retry_forever and secondary output shouldn't be exclusive. Indeed, I expect Fluentd to retry forever in case of recoverable errors, but to proceed with secondary in case of unrecoverable errors. In my understanding, the code already works like this and the only thing to do is to remove the configuration constraint (to confirm by someone more familiar with Fluentd's code).

[repeatedly]

Looks good to me. I will check the code.

[repeatedly]

Patch is here: #2276

[yann-soubeyrand]

Thanks a lot!