markmcdowell/NLog.Targets.ElasticSearch

Client Bulk Sync --> Async

Msuphi opened this issue · 2 comments

Hi,
Why did we not do it asynchronously, is there a specific reason?
Wouldn't it cause slowness when something like BufferSize = 250 ?

var result = _client.Bulk(payload); --> var result = _client.BulkAsync(payload);

Thanks ..

Why did we not do it asynchronously, is there a specific reason?

Think it is because NLog AsyncTaskTarget is somewhat new (introduced with NLog 4.6).

stale commented

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.