search

mattbaird / elastigo

A Go (golang) based Elasticsearch client library.
Apache License 2.0
943 stars 241 forks source link

Added request tracing support to BulkIndexer #201

Closed pavel-main closed 9 years ago

pavel-main commented 9 years ago

See last sentence of #194

weberr13 commented 9 years ago

@kpashka Can we get some basic tests around this? Currently the code is "safe" not having RequestTracer set automatically and it is up to the user to set their own tracer (it is public, so they can do that). If we introduce this method we start excercising code in baserequest.go that wasn't being used previously so tests would be imperative before merging.

pavel-main commented 9 years ago

@weberr13 I will make new PR, lost previous fork