ElasticSearchIntegration - `performRequest` does not exist on class about dd-trace-php HOT 2 CLOSED

Hi @Lavoaster sorry to hear of the problem and thanks for taking the time to open this pull request.

From my own debugging I can see that we rely on 5.3.* of the elasticsearch/elasticsearch package, however, the composer.json file in here suggests that you rely on the 1.2.* version. (over 4 years old)

We have plans to support recent versions of ES.

It appears that the method you want to hook into doesn't exist in later version of the package. I'm not entirely sure, otherwise I'd send in a PR, but I feel like you're looking for this method https://github.com/elastic/elasticsearch-php/blob/master/src/Elasticsearch/Transport.php#L93.
If possible as well can you put instrumentation setup through try/catches and disable those that fail, possibly with an error somewhere?

The main point here is that we should properly support version 5.x and before that we have to provide a way to test multiple versions of the same library. You can follow this issue #185 as @inverse proposed a very good approach in my opinion.

I've had to disable the elastic search integration provider due to issue.

Just to make sure you did not have to change any code: you can do this by DD_INTEGRATIONS_DISABLED=elasticsearch.

from dd-trace-php.

Hi @LavaToaster, we completely rearchitected our API and such issues, like class/method not found cannot happen anymore by design. What we call 'sandboxed' instrumentation. I am closing this issue. Please feel free to comment if this does not hold true for your case and I will reopen it.

from dd-trace-php.