This quickfix ensures that we don't walk all of the announcements while looking for a specific appID/gatewayID.
Changes
Directly access map instead of using loop
Testing
Unit tests.
Notes for Reviewers
I lost the pprof dump but this is on the broker hot-path on both uplink and activation.
Are the seen sets still needed ?
Checklist
[x] Scope: The referenced issue is addressed, there are no unrelated changes.
[x] Compatibility: The changes are backwards compatible with existing API, storage, configuration and CLI, according to the compatibility commitments in README.md for the chosen target branch.
[ ] Documentation: Relevant documentation is added or updated.
[ ] Changelog: Significant features, behavior changes, deprecations and fixes are added to CHANGELOG.md.
[x] Commits: Commit messages follow guidelines in CONTRIBUTING.md, there are no fixup commits left.
Summary
This quickfix ensures that we don't walk all of the announcements while looking for a specific
appID
/gatewayID
.Changes
Testing
Unit tests.
Notes for Reviewers
I lost the
pprof
dump but this is on thebroker
hot-path on both uplink and activation.Are the
seen
sets still needed ?Checklist
README.md
for the chosen target branch.CHANGELOG.md
.CONTRIBUTING.md
, there are no fixup commits left.