Backpressure #4452
Backpressure #4452
Conversation
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
| /// Enables or disables automatic backpressure handling. When enabled, the SDK will monitor system health and | ||
| /// reduce the sampling rate of events and transactions when the system is under load. | ||
| /// </summary> | ||
| public bool EnableBackpressureHandling { get; set; } = false; |
There was a problem hiding this comment.
will it eventually be 'true' by default or just while we validate it? if this is disabled forever by default it wont be of much value
There was a problem hiding this comment.
Is the benefit mainly for Sentry here?
I think what the Java SDK did was have this optional initially, until they got some feedback from SDK users with high throughput scenarios, then they turned it on by default when they were happy with it.
There was a problem hiding this comment.
I'm happy with either .. enabling by default right away or later.
With net10.0 and our 6.0 Release on the horizon, we could do an opt-in for now, and enable per default in 6.0.
The "pain point" there is that we need to update the docs then again.
There was a problem hiding this comment.
I've added an issue to make sure we circle back to this (by v6.0 at the latest):
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
Resolves #2410