Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

mateusz-baluch-xtb
Copy link

@mateusz-baluch-xtb mateusz-baluch-xtb commented Mar 28, 2025

This commit fixes self-suppression exception when using preferSecondary and all standby servers are unavailable
Fix for issue: #677

Make sure that:

  • You have read the contribution guidelines.
  • You have created a feature request first to discuss your contribution intent. Please reference the feature request ticket number in the pull request.
  • You use the code formatters provided here and have them applied to your changes. Don't submit any formatting related changes.
  • You submit test cases (unit or integration tests) that back your changes.

Issue description

Issue described in: #677

New Public APIs

NA

Additional context

NA

This commit fixes self-suppression exception when using preferSecondary and all standby servers are unavailable
@mp911de mp911de modified the milestones: 1.1.0.RELEASE, 1.0.8.RELEASE Oct 6, 2025
@mp911de mp911de changed the title GH-677 Fix self-suppression when standby is unavailable Fix self-suppression when standby is unavailable Oct 6, 2025
@mp911de mp911de added the type: bug A general bug label Oct 6, 2025
@mp911de mp911de linked an issue Oct 6, 2025 that may be closed by this pull request
mp911de pushed a commit that referenced this pull request Oct 6, 2025
…lf-suppression exception when using preferSecondary and all standby servers are unavailable.

[#678][resolves #677]
@mp911de mp911de closed this in bbdf0ea Oct 6, 2025
mp911de added a commit that referenced this pull request Oct 6, 2025
Use parameter injection to avoid resource control in tests.

[#678][#677]
mp911de pushed a commit that referenced this pull request Oct 6, 2025
…lf-suppression exception when using preferSecondary and all standby servers are unavailable.

[#678][resolves #677]
mp911de added a commit that referenced this pull request Oct 6, 2025
Refine exception propagation by collecting exceptions as list instead of using an atomic reference.
Also, introduce exception aggregator for easier exception surfacing.

[resolves #678][#677]
mp911de added a commit that referenced this pull request Oct 6, 2025
Use parameter injection to avoid resource control in tests.

[#678][#677]
@mp911de mp911de self-assigned this Oct 6, 2025
@mp911de
Copy link
Collaborator

mp911de commented Oct 6, 2025

Thank you for your contribution. That's merged, polished, and backported now.

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

Labels

type: bug A general bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Self-suppression exception when using preferSecondary

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.