appsignal-nodejs icon indicating copy to clipboard operation
appsignal-nodejs copied to clipboard

ElasticSearch instrumentation

Open luismiramirez opened this issue 2 years ago • 3 comments

https://github.com/aspecto-io/opentelemetry-ext-js/tree/master/packages/instrumentation-elasticsearch

Do we want to add this one to the default instrumentations? Note that it's not from the official contrib repository.

To do

  • [ ] Add extractor
  • [ ] Add integration test app

luismiramirez avatar Nov 10 '22 09:11 luismiramirez

:heavy_check_mark: All good!

New issue guide | Backlog management | Rules | Feedback

backlog-helper[bot] avatar Nov 10 '22 09:11 backlog-helper[bot]

We should have a conversation about whether we can/should add these ones. We may choose to support them by adding an extractor, but make them opt-in. If we do, the Sequelize and TypeORM ones are also very interesting.

unflxw avatar Nov 11 '22 08:11 unflxw

Agreed with having the conversation.

luismiramirez avatar Nov 11 '22 09:11 luismiramirez