getsentry / getsentry/sentry-python

StrawberryIntegration slows down query performance dramatically

Đang mở
#3,670 19 bình luận 0 reaction 1 người được giao Được @sentrivana nhận Xem trên GitHub
Better Async Support Improvement Integration: Django Integration: Strawberry Python
Ngôn ngữ chính
Python
Star
2.2k
Fork
669
Merge trung bình
1 ngày 1 giờ
Pull request đã merge (30 ngày)
213

Mô tả

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

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.