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

Support combining several DittoProtocol commands in a single aggregating one #289

Open
thjaeckle opened this issue Nov 20, 2018 · 3 comments

Comments

@thjaeckle
Copy link
Member

In #288 an alternative solution for doing multiple "modify" operations by only issuing a single DittoProtocol command was suggested:

Adding a DittoProtocol command which contains several other DittoProtocol commands (as simple Json array in its JSON form).

As this would be simpler than #288 (adding a "PATCH" command), we track it separately.

@avgustinmm
Copy link

avgustinmm commented Nov 26, 2018

It seems like something that we could use. In our case we get batch of change events from the gateways that we pass to ditto (through hono) using a hono message per command. A batch command for ditto means that we could send a single message per multiple commands that should optimize the hono usage (e.h. tragic, latency ..). That, in theory, could allow the ditto service to use the underlying database in more efficient way.

@thjaeckle thjaeckle added the community-interest Issues which were explicitly asked for by the Ditto community. label Nov 26, 2019
@yufei-cai
Copy link
Contributor

Is this relevant after #288?

@thjaeckle
Copy link
Member Author

It could still be, but with even lower priority when #288 is in place.

@thjaeckle thjaeckle removed the community-interest Issues which were explicitly asked for by the Ditto community. label Sep 21, 2022
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

No branches or pull requests

3 participants