Skip to content

chore(i18n): remove i18n string URL tracker#73370

Merged
artem-vavilov merged 1 commit into
stagingfrom
i18n-tracker-removal
Jun 22, 2026
Merged

chore(i18n): remove i18n string URL tracker#73370
artem-vavilov merged 1 commit into
stagingfrom
i18n-tracker-removal

Conversation

@artem-vavilov

@artem-vavilov artem-vavilov commented Jun 22, 2026

Copy link
Copy Markdown
Member

Removes the legacy i18n string URL tracking.

The tracker was originally added to record which i18n string keys appeared on which URLs and send that data to Firehose/Redshift for localization reporting. This is no longer needed now that we use LocalizeJS and have removed the legacy Crowdin sync pipeline.

Removing this code also eliminates a potential source of memory growth. The tracker keeps a singleton in-process buffer with a max logical size of 250 MB and flushes only every 8–16 hours. Since the buffer stores nested Ruby objects, its actual RSS/PSS impact can be much higher than 250 MB and matches the observed growth in Ruby objects and live Ruby heap slots.

Related PRs

This comment was marked as outdated.

chatgpt-codex-connector[bot]

This comment was marked as outdated.

@artem-vavilov artem-vavilov force-pushed the i18n-tracker-removal branch from 87fe5ff to 87cfec1 Compare June 22, 2026 11:18
@artem-vavilov artem-vavilov requested a review from a team June 22, 2026 12:02

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 44 out of 44 changed files in this pull request and generated no new comments.

@sureshc sureshc left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you! 🙏🏾

@artem-vavilov artem-vavilov merged commit baa3fcb into staging Jun 22, 2026
9 checks passed
@artem-vavilov artem-vavilov deleted the i18n-tracker-removal branch June 22, 2026 17:50
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.

5 participants