serilog-sinks-elasticsearch: Allow to forget Elasticsearch shipping errors.

Hi.

In the context of a particular application, where I am using this sinks to store log events in durable mode, happens that if there is any error on my Elasticsearch server I don’t want my app to have any issue of any kind because of it. If the Elastic server remains off for a long time then this sink continues saving the messages on disk and retrying to send them all: when the Elasctic server comes alive then all the messages are sent and this means consuming CPU and afecting my app at the end. For me it is peferible to lose messages rather than assuring to save them all having a CPU consumption peak.

Could it be possible just to add a new parameter in the options to say that I prefer to lose messages rather than retrying?

It should be as easy as modifying the file “ElasticSearchLogShipper.cs” on the following peace of code:

                   `if (count > 0)
                    {
                        var response = _state.Client.Bulk<DynamicResponse>(payload);

                        if (response.Success)  // <---- here would come the change
                        {
                            WriteBookmark(bookmark, nextLineBeginsAtOffset, currentFilePath);
                            _connectionSchedule.MarkSuccess();
                        }
                        else
                        {
                            _connectionSchedule.MarkFailure();
                            SelfLog.WriteLine("Received failed ElasticSearch shipping result {0}: {1}", response.HttpStatusCode, response.OriginalException);
                            break;
                        }
                    }`

Thank vou very much for your time. Jose