Skip to content

impl(pubsub): update last extension for successful extend#5530

Merged
PhongChuong merged 2 commits intogoogleapis:mainfrom
PhongChuong:subLastExtensionEvent
Apr 27, 2026
Merged

impl(pubsub): update last extension for successful extend#5530
PhongChuong merged 2 commits intogoogleapis:mainfrom
PhongChuong:subLastExtensionEvent

Conversation

@PhongChuong
Copy link
Copy Markdown
Contributor

@PhongChuong PhongChuong commented Apr 24, 2026

Lease state propagate extension results back to LeaseLoop using by the LeaseEvent ExtendCompleted and ExtendCompletedEO. The result are used to update the last extension value in the lease state.

Fixes: #5048

@product-auto-label product-auto-label Bot added the api: pubsub Issues related to the Pub/Sub API. label Apr 24, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 24, 2026

Codecov Report

❌ Patch coverage is 97.40260% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 97.75%. Comparing base (8ca77e7) to head (2b1d950).
⚠️ Report is 7 commits behind head on main.

Files with missing lines Patch % Lines
src/pubsub/src/subscriber/lease_state.rs 93.33% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5530      +/-   ##
==========================================
- Coverage   97.76%   97.75%   -0.01%     
==========================================
  Files         218      218              
  Lines       49498    49555      +57     
==========================================
+ Hits        48390    48443      +53     
- Misses       1108     1112       +4     

☔ 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.

@PhongChuong PhongChuong marked this pull request as ready for review April 27, 2026 14:52
@PhongChuong PhongChuong requested a review from a team as a code owner April 27, 2026 14:52
Comment thread src/pubsub/src/subscriber/lease_state.rs Outdated
Comment thread src/pubsub/src/subscriber/lease_state.rs Outdated
Comment thread src/pubsub/src/subscriber/lease_state.rs Outdated
@PhongChuong PhongChuong merged commit 0a84fba into googleapis:main Apr 27, 2026
35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: pubsub Issues related to the Pub/Sub API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve subscriber lease extension logic

2 participants