Skip to content
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

[IMPROVE] Improvements and fixes to stream source consumer creation on leadership change or on stream config source updates #4009

Merged
merged 2 commits into from Apr 5, 2023

Conversation

jnmoyne
Copy link
Contributor

@jnmoyne jnmoyne commented Mar 31, 2023

  • Tests added
  • Branch rebased on top of current main (git pull --rebase origin main)
  • Changes squashed to a single commit (described here)
  • Build is green in Travis CI
  • You have certified that the contribution is your original work and that you license the work to the project under the Apache 2 license

…eadership change or stream config update to work with the new stream sourcing

- Updates source index name format and adds two fields (source's filter and transform) to the `"Nats-Stream-Source"` message header
- Backwards compatibility for streams containing previous (2.9) message headers for seamless upgrade
- Update TestJetStreamSourceBasics as it could flap sometimes

Signed-off-by: Jean-Noël Moyne <jnmoyne@gmail.com>
@jnmoyne jnmoyne requested a review from a team as a code owner March 31, 2023 22:54
server/stream.go Outdated Show resolved Hide resolved
server/stream.go Outdated Show resolved Hide resolved
server/stream.go Outdated Show resolved Hide resolved
server/stream.go Outdated Show resolved Hide resolved
server/stream.go Outdated Show resolved Hide resolved
Signed-off-by: Jean-Noël Moyne <jnmoyne@gmail.com>
@derekcollison
Copy link
Member

This ready to review?

@jnmoyne
Copy link
Contributor Author

jnmoyne commented Apr 5, 2023

Apologies, yes it is!

@derekcollison derekcollison self-requested a review April 5, 2023 03:04
Copy link
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 4525bde into dev Apr 5, 2023
2 checks passed
@derekcollison derekcollison deleted the jnm/streamsourcefixes branch April 5, 2023 03:09
caspervonb pushed a commit to caspervonb/nats-server that referenced this pull request Apr 5, 2023
…n leadership change or on stream config source updates (nats-io#4009)

 - [X] Tests added
- [X] Branch rebased on top of current main (`git pull --rebase origin
main`)
- [X] Changes squashed to a single commit (described
[here](http://gitready.com/advanced/2009/02/10/squashing-commits-with-rebase.html))
 - [x] Build is green in Travis CI
- [X] You have certified that the contribution is your original work and
that you license the work to the project under the [Apache 2
license](https://github.com/nats-io/nats-server/blob/main/LICENSE)
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.

None yet

2 participants