Add some data about response data at ES instrumentation

[Sparkycz]

What does this pull request do?

Adds http.status_code and db.row_affected from ElasticSearch response in ES instrumentation

Related issues

closes #354

[ghost]

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

• Build Cause: Pull request #1108 updated

• Start Time: 2021-06-28T14:51:34.254+0000

• Duration: 26 min 10 sec

• Commit: 7e89b0c

Test stats 🧪

Test	Results
Failed	0
Passed	9481
Skipped	8604
Total	18085

Trends 🧪

💚 Flaky test report

Tests succeeded.

Expand to view the summary

Test stats 🧪

Test	Results
Failed	0
Passed	9481
Skipped	8604
Total	18085

[basepi]

Looks awesome, thanks for grabbing this!

[basepi]

I'm a bit mystified as to why the tests are failing. I'll keep digging but the result_data["hits"]["total"]["value"] should definitely be working based on my local testing.

[basepi]

I spoofed the tests to point to my local elasticsearch instance and they passed. I'm going to investigate the elasticsearch that's used in the integration tests (https://github.com/elastic/apm-integration-testing) -- I'm guessing the issue is there.

[basepi]

Oh, I figured it out. It's failing against Elasticsearch 2.x (which is not surprising). Just need to add some conditionals. Stay tuned.

[Sparkycz]

Thanks a lot for finishing the change. I’ve been out of Github for a while..