Skip to content

Fix Reliable Topic Exception Handling#1056

Open
emreyigit wants to merge 5 commits into
hazelcast:masterfrom
emreyigit:fix-rt-ex-handling
Open

Fix Reliable Topic Exception Handling#1056
emreyigit wants to merge 5 commits into
hazelcast:masterfrom
emreyigit:fix-rt-ex-handling

Conversation

@emreyigit

Copy link
Copy Markdown
Member

The TestClusterRestartWhenSubscribed is acting flaky. By suspecting ShouldDisposeAsync to compared to Java counterpart, have changed the tail sequence to head sequence. Semantically, head should be the tail +1 but hoping to fix the flakiness with this change.

@emreyigit emreyigit changed the base branch from master to 5.7.z May 18, 2026 10:03
@codecov

codecov Bot commented May 18, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 82.65%. Comparing base (d04e048) to head (d622b60).

Files with missing lines Patch % Lines
...DistributedObjects/ReliableTopicMessageExecutor.cs 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1056      +/-   ##
==========================================
- Coverage   82.69%   82.65%   -0.05%     
==========================================
  Files        1060     1060              
  Lines       25834    25834              
==========================================
- Hits        21364    21353      -11     
- Misses       4470     4481      +11     

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

@emreyigit emreyigit changed the base branch from 5.7.z to master May 21, 2026 08:48
@emreyigit emreyigit requested a review from ihsandemir June 1, 2026 07:53
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.

1 participant