feat(spanner): add otel semconv sql statement#12631
feat(spanner): add otel semconv sql statement#12631yordis wants to merge 3 commits intogoogleapis:mainfrom
Conversation
feat(spanner): add otel semconv to spanner
4d68dd6 to
08ea328
Compare
|
@rahul2393 could you help me with this one? |
|
I created PR with more attributes #11652 |
|
@yordis you can apply the change in your GORM application by running this |
|
@rahul2393 I already tested this in prod, but we had to rollback due to the streaming issue
|
|
Ok the context of this PR was to add sql statement in OTEL traces right? |
Correct
I am not disagreeing here, but taking this PR reduce the blast radious, to me, whatever makes it to production ASAP; since we are blind right now. I rather have something, than nothing |
|
I have asked my team to review my PR since anyways we were planning to merge that for the long time. |
|
Closing this in favor of #11652 |

No description provided.