Skip to content

Conversation

@TNBcrpt
Copy link

@TNBcrpt TNBcrpt commented Oct 24, 2024

Before submitting a pull request, please take a look at our
Contributing guidelines and verify:

  • If you've added code that should be tested, please add tests.
  • Ensure your code lints and the test suite passes (yarn lint) & (yarn test).

billyvg and others added 13 commits October 15, 2024 13:29
…eta.0

meta: Prepare prelease of 8.35.0-beta.0
…y#14014)

This is not intended to be merged to prod, I would like to debug how/why
we keep hitting EventBuffer limits on Sentry SaaS. Aside from debugging
statements, there should be no other behavioral changes.

---------

Co-authored-by: Catherine Lee <[email protected]>
Re-adds replay SDK debug statements from a prior prelease (and includes 8.35.0 changes).
… as well (getsentry#14069)

Removes a log statement that is output when buffer is already full
(which will keep overflowing the buffer). Also noticed a log statement
in `stop()` that would trigger when we stop due to event buffer being
full, which would compoound the issue. This last fix should be pulled
into the SDK.
Not intended for public use.

* debug(replay): Remove noisy log that is causing event buffer overflow as well ([getsentry#14069](getsentry#14069))
@TNBcrpt TNBcrpt requested review from a team as code owners October 24, 2024 05:08
@billyvg billyvg closed this Oct 24, 2024
@billyvg
Copy link
Member

billyvg commented Oct 24, 2024

bot?

@TNBcrpt
Copy link
Author

TNBcrpt commented Oct 24, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants