Skip to content

Set ttl messge property#62

Merged
emilyzheng merged 1 commit intorelease-engineering:mainfrom
emilyzheng:fix-expiry
Jan 27, 2026
Merged

Set ttl messge property#62
emilyzheng merged 1 commit intorelease-engineering:mainfrom
emilyzheng:fix-expiry

Conversation

@emilyzheng
Copy link
Copy Markdown
Contributor

Set property ttl: time to live of the message measured in seconds.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jan 27, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (a2858ca) to head (ae22b76).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #62   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           24        24           
  Lines         1276      1277    +1     
=========================================
+ Hits          1276      1277    +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Set property ttl: time to live of the message measured in seconds.
@emilyzheng emilyzheng changed the title Update expiry header Set ttl messge property Jan 27, 2026
Copy link
Copy Markdown

@lslebodn lslebodn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

BTW here is related part from ActiveMQ doc:

Messages can be set with an optional time to live when sending them.Apache ActiveMQ Artemis will not deliver a message to a consumer after it’s time-to-live has been exceeded.

If the message hasn’t been delivered by the time that time-to-live is reached the server can discard it.

@emilyzheng emilyzheng merged commit c818e39 into release-engineering:main Jan 27, 2026
7 checks passed
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