Skip to content

Interface for serialized messages#618

Closed
atimin wants to merge 2 commits intoros2-rust:mainfrom
reductstore:main
Closed

Interface for serialized messages#618
atimin wants to merge 2 commits intoros2-rust:mainfrom
reductstore:main

Conversation

@atimin
Copy link
Copy Markdown

@atimin atimin commented Mar 27, 2026

We're developing a bridge that will receive and store serialised data in its original form in a database. This patch enables that functionality. I didn't put much effort into it as it's generated code. However, if the team sees value in this feature, I can develop it further.

Thank you for the awesome library!

@atimin
Copy link
Copy Markdown
Author

atimin commented Mar 27, 2026

Looks like there is another PR doing the same: #592. Please, ignore this one.

@atimin atimin closed this Mar 27, 2026
@esteve
Copy link
Copy Markdown
Collaborator

esteve commented Mar 28, 2026

@atimin no problem! Were you able to try the approach in #592 ? Does it work for you?

@atimin
Copy link
Copy Markdown
Author

atimin commented Apr 1, 2026

Hey @esteve, I tried to adopt the code from #592. It doesn't work in my environment. I need more time to investigate why. It may be the way I've integrated it.

However, I can see that my patch adheres to the project API style and uses callbacks. I've already tested it in a production environment, processing thousands of messages per second with no issues. Would you prefer to use my work?

In any case, I'm open to collaborating on both approaches.

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