-
Notifications
You must be signed in to change notification settings - Fork 170
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
TracingDriverConnectionFactory calls the deprecated method AbstractMySQLPlatform::getName() #728
Comments
No, it isn't. It's a completely different deprecation, which involves the connection class and not the driver class. Actually, I think that the only solution to the problem is to implement a |
Would we really need a |
Although not required, I tried to follow as much as possible the guidelines of OpenTelemetry regarding the Connection-level attributes, Since the specification is the best standard out of here in regards to distributed tracing, I think we should continue on this way. |
The list of Sentry's span operations can be found here. It's mandatory to use these, as certain expectations in the product rely on them. |
We're not talking about span operations, but about span tags 😃 |
I see. There is a movement away from creating tags automatically in the SDKs as well, I'll clarify internally. Most likely, we'll move this data to |
@cleptric any updates? |
We should move the span tag values over to span data. As Spans are not indexed in Sentry, this has no impact on the product. I would be OK to remove |
I still don't understand what this has to do with this issue. Note I'm not arguing about whether it makes sense or not, but just about the fact that it's a completely different topic.
It was added because I was trying to follow the OpenTelemetry specification which requires this info to be available. Yes, maybe it doesn't add much value, but sometimes respecting some kind of standard is valuable 😉 |
PR is up #743 |
@ste93cry we try to follow the otel conventions for span attributes as much as possible in span data. This is documented here. Our span data conventions are essentially a superset of otel's, because they are lacking some conventions for mobile/browser and also don't have everything documented for the web server use case also. |
I assume that your answer is about moving the information from tags to data. As I said before, I don't argue about doing it, to be honest I care little about it, I was just pointing out that the problem being reported in this issue has nothing to do with such change, therefore we should just care about solving the deprecation in the SDK. Any other thing should be done in a different PR. |
Environment
Steps to Reproduce
AbstractMySQLPlatform::getName()
was deprecated in doctrine/dbal#4749 but is still used inSentry\SentryBundle\Tracing\Doctrine\DBAL\TracingDriverConnectionFactory
.This triggers the following deprecation warning:
Doctrine team suggests to use the class name to identify the platform instead.
As far as I can see the only usage of the platform name is in
Sentry\SentryBundle\Tracing\Doctrine\DBAL\TracingDriverConnection::getSpanTags()
.The text was updated successfully, but these errors were encountered: