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

pubsub: Memory growth observed while sending messages with ordering keys #4288

Closed
srikanthvukkadapu opened this issue Jun 19, 2021 · 1 comment · Fixed by #4282
Closed

pubsub: Memory growth observed while sending messages with ordering keys #4288

srikanthvukkadapu opened this issue Jun 19, 2021 · 1 comment · Fixed by #4282
Assignees
Labels
api: pubsub Issues related to the Pub/Sub API. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.

Comments

@srikanthvukkadapu
Copy link

srikanthvukkadapu commented Jun 19, 2021

Client

PubSub v1.10.1

Environment

Ubuntu 20.04

Go Environment

go 1.16

Expected behavior

memory growth should not be observed while sending messages using transient ordering keys (ordering keys which have a limited life span)

Actual behavior

Memory growth is observed while sending messages using Topic.Publish method with ordering keys. Attaching the proof report image. It looks like the maps s.bundlers and s.outstanding of PublishScheduler are growing.

Should the increment of outstanding s.outstanding[key]++ be done when the key is found in the map only as it is already initialised to 1 when it is not found in the map.

image

@srikanthvukkadapu srikanthvukkadapu added the triage me I really want to be triaged. label Jun 19, 2021
@product-auto-label product-auto-label bot added the api: pubsub Issues related to the Pub/Sub API. label Jun 19, 2021
@hongalex hongalex added priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns. and removed triage me I really want to be triaged. labels Jun 21, 2021
@hongalex
Copy link
Member

Someone else actually pointed out this issue and will be resolved by #4282.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: pubsub Issues related to the Pub/Sub API. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. type: bug Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants