sentry-rust icon indicating copy to clipboard operation
sentry-rust copied to clipboard

slow http/db ops UI question

Open jessfraz opened this issue 2 years ago • 5 comments

Environment

How do you use Sentry? Sentry SaaS (sentry.io)

Which SDK and version? Sentry-rust master

So I have all these http.server db.sql.query etc spans/transactions but they don't show up in the UI under slow http/db ops

See below Screen Shot 2022-03-15 at 9 10 00 AM

I tried looking at the source code for the js client and it seems like I'm doing all the same stuff. I tried looking at the UI code and it seems like it should work for http and db ops so I guess I'm wondering do I need to explicitly call it that versus http.server or an I missing some other param?

Any help is appreciated

jessfraz avatar Mar 15 '22 16:03 jessfraz

My secondary question is do you all have a discord or IRC channel you hang out in because I have a few other questions but don't want to spam your issues

jessfraz avatar Mar 15 '22 16:03 jessfraz

the difference actually seems to be a breadcrumb with category http or db in each, so is that necessary?

jessfraz avatar Mar 15 '22 17:03 jessfraz

do you all have a discord or IRC channel you hang out in

Our discord server is linked from the Readme. You can find me and the rest of the team in the rust channel and a few others.

As for the actual question, I will forward this to someone else who might be able to help you with that.

Swatinem avatar Mar 25 '22 10:03 Swatinem

@Swatinem I was digging into sentry performance for docs.rs itself, and was missing the sql queries too.

Is it perhaps enough to set an specially crafted tracing-span that could be picked up by the sentry SDK?

or is this special behaviour in the SDK that I would have to call separately?

syphar avatar Mar 04 '24 13:03 syphar

Is it perhaps enough to set an specially crafted tracing-span that could be picked up by the sentry SDK?

Yes, pretty much you would need op: db and description: RAW SQL. that is what other SDKs with auto-instrumentation are doing.

Some problems might be:

  • The #[tracing::instrument] integration automatically infers the op and description from the tracing span name.
  • Not relevant for the Rust SDK in its current form, but there was talk about deprecating the difference between op/description at some point, though I don’t yet know how that would change this auto-classification.

Swatinem avatar Mar 04 '24 14:03 Swatinem