-
Notifications
You must be signed in to change notification settings - Fork 23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add advisories for pull consumer groups #299
Conversation
b5a7e02
to
49ed0c9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some minor comments
adr/ADR-42.md
Outdated
const JSStreamGroupPinnedAdvisoryType = "io.nats.jetstream.advisory.v1.consumer_group_pinned" | ||
|
||
// JSStreamGroupPinnedAdvisory that a group switched to a new pinned client |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
// JSStreamGroupPinnedAdvisory that a group switched to a new pinned client | |
// JSStreamGroupPinnedAdvisory indicates that a group switched to a new pinned client |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We are also missing the Advisory subject from the PR.
JSAdvisoryConsumerPinnedPre`` = "$JS.EVENT.ADVISORY.CONSUMER.PINNED
JSAdvisoryConsumerUnpinnedPre = "$JS.EVENT.ADVISORY.CONSUMER.UNPINNED
?
Signed-off-by: R.I.Pienaar <[email protected]>
49ed0c9
to
c523e34
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
No description provided.