supabase-community / sentry-integration-js

MIT License
9 stars 3 forks source link

Adapt Span data to take advantage of the queries view in Sentry Performance #5

Open smeubank opened 6 months ago

smeubank commented 6 months ago

Feature request

As span data is set today, it will not appear in Sentry's Query Module.

image

Describe the solution you'd like

Span's must be set in the required format to display in the query module.

Describe alternatives you've considered

We should comply as much as possible from a client side PoV but it may also be reasonable to add db.supabase similar to db.prisma

Additional context

Specifically Operation should be

smeubank commented 6 months ago

in the case of Prism in relay, Sentry accepts a special case

from Kamil

For supabase there is no raw SQL query, so they use https://postgrest.org/en/v12/index.html

smeubank commented 6 months ago

@kamilogorek what kind of cardinality should we expect with PostgREST

Is it parameterized by nature or within the integration in anyway?

smeubank commented 6 months ago

@kamilogorek fyi https://github.com/getsentry/relay/pull/3153/

kamilogorek commented 6 months ago

@smeubank following up on this, it looks like there's no changes required now on the integration side?