-
Notifications
You must be signed in to change notification settings - Fork 24.8k
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
Add CBOR-friendly bulk format #8481
Comments
Any updates? CBOR would be great compliment to improved number handling in 5.0 release considering it has variable size numerics. |
Since CBOR allows embedded CBOR data item, maybe we could have a CBOR specific bulk format that is basically an array of data items but use https://tools.ietf.org/html/rfc7049#section-2.4.4.1 for the body item? |
@tlrx that's an interesting idea... not sure how easy it'd be to fit into our bulk parsing code. Please could you investigate when you have some time? |
Two more options how bulk requests could be achieved:
|
The current bulk format in JSON uses the
\n
as end-of-line markers, allowing the coordinating node to parse just the metadata line, then skip over the body line.CBOR doesn't have any disallowed characters or character sequences, so this mechanism will not work. Instead we should have a CBOR specific format which looks like this:
The
int_length_of_body_line
would allow us to skip to the next metadata line.For the
delete
action, noint_length_of_body_line
would be required, as we already know that it can't have a body.Depends on #7640
The text was updated successfully, but these errors were encountered: