getsentry / getsentry/sentry-python

StrawberryIntegration slows down query performance dramatically

Offen
#3,670 19 Kommentare 0 Reaktionen 1 zugewiesene Person Beansprucht von @sentrivana Auf GitHub ansehen
Better Async Support Improvement Integration: Django Integration: Strawberry Python
Vorherrschende Sprache
Python
Sterne
2.2k
Forks
669
Ø Merge
1 T. 1 Std.
Gemergte PRs (30 T.)
213

Beschreibung

### How do you use Sentry?

Sentry Saas (sentry.io)

### Version

2.17.0

### Steps to Reproduce

1. Set up a Django project with Strawberry async.
2. Set up a query for an endpoint that returns many objects. I tested with 2,000 objects and only 2 fields for each - an ID and a string, e.g.

```python
@strawberry_django.type(models.Item, fields=["id", "value"], pagination=True)
class Item:
pass

@strawberry.type
class Query:
items: list[types.Item] = strawberry_django.field()
```

```graphql
query Items {
items(pagination: { limit: 2000 }) {
id
value
}
}
```

### Expected Result

Not much performance difference with or without `StrawberryIntegration`

### Actual Result

With `StrawberryIntegration`, this took 3-4x longer. In my case, about 800ms. Without it, I got the results back in 200-250ms.

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.