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.

Sign up for GitHub

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

Jump to bottom

Add correlationId to message #116

Closed
pshemass opened this issue Nov 26, 2019 · 4 comments
Closed

Add correlationId to message #116

pshemass opened this issue Nov 26, 2019 · 4 comments
Labels
enhancement New feature or request good first issue Good for newcomers
Milestone
0.1.0

Comments

@pshemass
Copy link
Contributor

All messages in the cluster should have identifier. When message are part of the same "transaction" they should have the same prefix in order to group them together.
@mijicd mijicd added enhancement New feature or request good first issue Good for newcomers labels Dec 3, 2019
@mijicd mijicd added this to the 0.1.0 milestone Dec 13, 2019
@klangner
Copy link
Contributor

Do I understand it correctly that I should add id to the class Message to implement it?
Of course this will require changes in serialization and probably other part.
This id should have String type?

@mijicd
Copy link
Member

mijicd commented Jan 24, 2020

@klangner exactly. It can be UUID or any alternative identifier you propose.

@klangner
Copy link
Contributor

I will work on it then.

@mijicd
Copy link
Member

mijicd commented Feb 16, 2020

Closed in #160

@mijicd mijicd closed this as completed Feb 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Milestone
0.1.0
Development

No branches or pull requests
3 participants
@klangner @pshemass @mijicd