Skip to content

NRG: Support for replying to forwarded proposals#6157

Merged
derekcollison merged 1 commit intomainfrom
neil/nrgforwardproposal
Nov 22, 2024
Merged

NRG: Support for replying to forwarded proposals#6157
derekcollison merged 1 commit intomainfrom
neil/nrgforwardproposal

Conversation

@neilalexander
Copy link
Copy Markdown
Member

This adds support for reply subjects on forwarded proposals, so we can know whether or not a leader has acted upon those proposals yet.

The ForwardProposal function does NOT yet use this functionality as we cannot yet know in a mixed-version or upgrade scenario yet if the remote side will be able to respond.

Signed-off-by: Neil Twigg neil@nats.io

This adds support for reply subjects on forwarded proposals, so we can
know whether or not a leader has acted upon those proposals yet.

The `ForwardProposal` function does NOT yet use this functionality as
we cannot know in a mixed-version or upgrade scenario yet if the remote
side will be able to respond.

Signed-off-by: Neil Twigg <neil@nats.io>
@neilalexander neilalexander requested a review from a team as a code owner November 21, 2024 11:18
@derekcollison derekcollison self-requested a review November 22, 2024 15:20
Copy link
Copy Markdown
Member

@derekcollison derekcollison left a comment

Choose a reason for hiding this comment

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

LGTM

@derekcollison derekcollison merged commit 972646f into main Nov 22, 2024
@derekcollison derekcollison deleted the neil/nrgforwardproposal branch November 22, 2024 15:21
neilalexander added a commit that referenced this pull request Nov 22, 2024
Includes:

- #6147
- #6150
- #6151
- #6153
- #6154
- #6146
- #6139
- #6152
- #6157
- #6161

Signed-off-by: Neil Twigg <neil@nats.io>
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.

2 participants