Skip to content

Conversation

@bjtitus
Copy link
Contributor

@bjtitus bjtitus commented Dec 28, 2025

This is causing unnecessary CPU usage when logging is very active. The tracing data Firebase collects (presumably stack traces) is quite large and leads to these events being processed:

CleanShot 2025-11-18 at 16 24 27@2x

The tracing is available in the Firebase admin but I don't believe we have used it in a couple years.

To test

  • Check FileLog messages with the in-app log viewer to make sure logging is working

Checklist

  • I have considered if this change warrants user-facing release notes and have added them to CHANGELOG.md if necessary.
  • I have considered adding unit tests for my changes.
  • I have updated (or requested that someone edit) the spreadsheet to reflect any new or changed analytics.

This is causing unnecessary network activity
@bjtitus bjtitus added the [Type] Tech Debt Applies to issues involving upgrades or refactoring to maintain or enhance the codebase. label Dec 28, 2025
@bjtitus bjtitus added this to the 8.3 milestone Dec 28, 2025
@bjtitus bjtitus marked this pull request as ready for review January 2, 2026 01:21
@bjtitus bjtitus requested a review from a team as a code owner January 2, 2026 01:21
@bjtitus bjtitus requested a review from SergioEstevao January 2, 2026 01:21
Copy link
Contributor

@SergioEstevao SergioEstevao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@bjtitus bjtitus merged commit ac988be into trunk Jan 5, 2026
9 of 11 checks passed
@bjtitus bjtitus deleted the bjtitus/remove-log-trace branch January 5, 2026 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Type] Tech Debt Applies to issues involving upgrades or refactoring to maintain or enhance the codebase.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants