Skip to content

Base unknown ttl on req timeout - #771

Open
yashnevatia wants to merge 5 commits into
mainfrom
base-unknownTTL-on-reqTimeout
Open

yashnevatia wants to merge 5 commits into
mainfrom
base-unknownTTL-on-reqTimeout

Conversation

@yashnevatia

@yashnevatia yashnevatia commented Sep 16, 2026

Copy link
Copy Markdown
Contributor
  • Bump capcommon and chainconsensus packages to import new helpers and handler
  • Set defaultMaxUnknownRequestsCacheSize to 1000.
    • Each cache entry is tied to one requestID which is tied to one execID (and query number).
    • Each execID is one obersvation
    • Currently, max observation size is 5kb.
    • So max cache memory footprint is 5 MB
    • Average max requestTimeout is 60s. On average, TTL will be set as 120s.
    • So this cache will use up to 5 MB of data if we receive more than 100 unknown requests within 120s.

@github-actions

Copy link
Copy Markdown
Contributor

👋 yashnevatia, thanks for creating this pull request!

To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team.

Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks!

dhaidashenko
dhaidashenko previously approved these changes Sep 17, 2026
@cl-sonarqube-production

Copy link
Copy Markdown

Quality Gate failed Quality Gate failed

Failed conditions
14.8% Duplication on New Code (required ≤ 5%)

See analysis details on SonarQube

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.

2 participants