Skip to content

Conversation

popduke
Copy link
Contributor

@popduke popduke commented Sep 17, 2025

This PR addresses an issue where route additions or removals trigger invalidation of all cached entries matching the topic filter rules, potentially causing performance jitter when the topic filter contains wildcards.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 17824084634

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 173 unchanged lines in 20 files lost coverage.
  • Overall coverage increased (+0.4%) to 63.745%

Files with Coverage Reduction New Missed Lines %
org/apache/bifromq/basekv/raft/RaftNodeStateFollower.java 1 82.62%
org/apache/bifromq/dist/worker/cache/SubscriptionCache.java 1 96.36%
org/apache/bifromq/dist/worker/schema/GroupMatching.java 1 86.67%
org/apache/bifromq/mqtt/handler/MQTTPersistentSessionHandler.java 1 74.51%
org/apache/bifromq/starter/utils/ClusterDomainUtil.java 1 75.0%
org/apache/bifromq/basekv/raft/RaftNodeStateLeader.java 2 85.23%
org/apache/bifromq/dist/worker/cache/TenantRouteMatcher.java 2 94.62%
org/apache/bifromq/inbox/server/InboxFetchPipeline.java 2 57.21%
org/apache/bifromq/plugin/eventcollector/distservice/GroupFanoutThrottled.java 2 0.0%
org/apache/bifromq/plugin/eventcollector/distservice/PersistentFanoutThrottled.java 2 0.0%
Totals Coverage Status
Change from base Build 17737460901: 0.4%
Covered Lines: 24433
Relevant Lines: 37000

💛 - Coveralls

@popduke popduke merged commit f62f6d3 into apache:main Sep 19, 2025
4 checks passed
@popduke popduke deleted the feat-route-cache-improvement branch September 19, 2025 01:26
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.

3 participants