-
Notifications
You must be signed in to change notification settings - Fork 26
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
Enhance BatchRequestCore (and implement node-side equivalent) #243
Comments
This could include the whole record batch management, as well (on both sides). |
It should use the new record batcher, too. |
Should probably include sociomantic-tsunami/dlsproto#84 by default. |
Btw, I the code pretty much ready, I'll clean it up and submit a PR in the next few days. I've successfully converted DMQ's Consume request and DLS's GetRange reqeuest to use it. |
Oh, yes, I think it should include that feature by default (it also includes double-buffering). |
Should use https://github.com/sociomantic-tsunami/swarm/blob/v4.x.x/src/swarm/neo/util/Batch.d internally (at the request's option?). |
Feature: extend |
:( |
Moved to v5.5.0. |
The existing request core is fairly minimal. I'm sure there's a lot more common code that could be added (basic fiber implementations, for example).
Ditto for the node side of the equation (where no core code exists atm).
The text was updated successfully, but these errors were encountered: