getsentry / getsentry/sentry-python

StrawberryIntegration slows down query performance dramatically

オープン
#3,670 コメント 19 件 リアクション 0 件 担当者 1 名 @sentrivana が担当を希望しています GitHub で見る
Better Async Support Improvement Integration: Django Integration: Strawberry Python
主要言語
Python
スター
2.2k
フォーク
669
平均マージ
1日 1時間
マージ済み PR(30日)
213

説明

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

コントリビューションガイド

コントリビューションガイドを開く

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。