Skip to content
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

fix: reduce ws meta tracking memory consumption #1720

Merged
merged 1 commit into from
Apr 14, 2024

Conversation

tkurki
Copy link
Member

@tkurki tkurki commented Apr 14, 2024

Fixes #1717.

Reduce memory consumption of string values used in tracking what metadata we have sent for a ws connection by keeping a cache of shared string values for this. This will deduplicate the values and help with memory consumption with a large number of AIS targets and multiple ws connections.

The cache is purged every 30 minutes to get rid of data for AIS targets that have disappeared.

@tkurki tkurki added the fix label Apr 14, 2024
@tkurki tkurki requested review from sbender9 and panaaj April 14, 2024 05:55
Fixes #1717.

Reduce memory consumption of string values used in tracking
what metadata we have sent for a ws connection by keeping
a cache of shared string values for this. This will deduplicate
the values and help with memory consumption with a large
number of AIS targets and multiple ws connections.

The cache is purged every 30 minutes to get rid of data
for AIS targets that have disappeared.
@tkurki tkurki force-pushed the fix-reduce-sentmeta-memory branch from 213cec1 to a47ee19 Compare April 14, 2024 09:51
@tkurki tkurki merged commit b4ca536 into master Apr 14, 2024
4 checks passed
@tkurki tkurki deleted the fix-reduce-sentmeta-memory branch April 14, 2024 09:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Excessive memory consumption of tracking sent metadata
2 participants