getsentry / getsentry/sentry-python

StrawberryIntegration slows down query performance dramatically

Ouverte
#3,670 19 commentaires 0 réactions 1 personne assignée Réclamée par @sentrivana Voir sur GitHub
Better Async Support Improvement Integration: Django Integration: Strawberry Python
Langage dominant
Python
Étoiles
2.2k
Forks
669
Merge moyen
1 j 1 h
PR mergées (30 j)
213

Description

### 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.

Guide de contribution

Ouvrir le guide de contribution

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.