-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Send event to Sentry when event can't be inserted to ES good index #167
Comments
Worth windowing this a bit - on a large pipeline this could be thousands of events failing! You might need some form of internal counter which flushes to Sentry every N minutes to protect the Sentry server from accidental DDoS. |
You're right. We're sending events to ES per batch, so we can send only one notification to Sentry per batch. |
Even per batch it could be a lot - the batch is determined by the sink parameters (time, bytes, record count) if I understand correctly. If your time-limit is say 100ms then you could get 10 failures per second coming to Sentry (extrapolating out to 864,000 per day at that rate if consistent). This is obviously a very pessimistic view but could eventuate... |
Do we pay for a maximum number of events sent to Sentry ? |
No - but we will need to scale up Sentry enormously if we send huge volume to it :) |
Oh we are self hosting it ! I had missed this bit ! |
On top of a bad row being emitted, we want to be notified when this happens to detect ASAP ES mapping failures with union types.
The text was updated successfully, but these errors were encountered: