Skip to content

CAMEL-23615: ErrorRegistry - disable TTL by default#23575

Merged
davsclaus merged 1 commit into
mainfrom
CAMEL-23615-error-registry-disable-ttl-default
May 27, 2026
Merged

CAMEL-23615: ErrorRegistry - disable TTL by default#23575
davsclaus merged 1 commit into
mainfrom
CAMEL-23615-error-registry-disable-ttl-default

Conversation

@davsclaus
Copy link
Copy Markdown
Contributor

Summary

  • Change the ErrorRegistry TTL default from 1 hour to 0 (disabled), so entries are retained based only on the maximum entries limit (100)
  • Guard TTL eviction logic to skip when TTL is zero or negative
  • Update documentation and generated catalog metadata

Rationale

Most error registries and similar in-memory stores (e.g., Micrometer, Spring Boot Actuator trace stores) either don't use time-based eviction or keep entries much longer than 1 hour. Since the registry is already bounded by maximumEntries=100, time-based eviction is unnecessary by default and can cause users to miss recent errors that were evicted too quickly.

Test plan

  • Existing ManagedErrorRegistryTest passes
  • Core modules build successfully
  • Catalog metadata regenerated

Claude Code on behalf of Claus Ibsen

The error registry now keeps entries based only on the maximum entries
limit (100) without time-based eviction. The TTL default is changed
from 1 hour to 0 (disabled). Users can still enable TTL by setting
camel.errorRegistry.timeToLiveSeconds to a positive value.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

🌟 Thank you for your contribution to the Apache Camel project! 🌟
🤖 CI automation will test this PR automatically.

🐫 Apache Camel Committers, please review the following items:

  • First-time contributors require MANUAL approval for the GitHub Actions to run
  • You can use the command /component-test (camel-)component-name1 (camel-)component-name2.. to request a test from the test bot although they are normally detected and executed by CI.
  • You can label PRs using skip-tests and test-dependents to fine-tune the checks executed by this PR.
  • Build and test logs are available in the summary page. Only Apache Camel committers have access to the summary.

⚠️ Be careful when sharing logs. Review their contents before sharing them publicly.

@davsclaus davsclaus merged commit c4a0fcd into main May 27, 2026
7 checks passed
@davsclaus davsclaus deleted the CAMEL-23615-error-registry-disable-ttl-default branch May 27, 2026 19:54
@github-actions
Copy link
Copy Markdown
Contributor

🧪 CI tested the following changed modules:

  • catalog/camel-catalog
  • core/camel-api
  • core/camel-base-engine
  • core/camel-main
  • docs

ℹ️ Dependent modules were not tested because the total number of affected modules exceeded the threshold (50). Use the test-dependents label to force testing all dependents.

Build reactor — dependencies compiled but only changed modules were tested (5 modules)
  • Camel :: API
  • Camel :: Base Engine
  • Camel :: Catalog :: Camel Catalog
  • Camel :: Docs
  • Camel :: Main

⚙️ View full build and test results

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant