You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
First of all this is awesome! Thanks a lot for the package.
I haven't looked heavily yet but the msgp package seems to decode roughly ~70% faster given the same structure, which seems a little odd since one goal of capnproto is to reduce the allocations (and msgp has to construct everything).
I was thinking maybe splitting the framing out of the decoder so that Unmarshal can be used, allowing the user to decide whether or not it's safe to reference the memory. My main use-case for analytical type stuff where you just want to iterate and grab a field or two in-place.
Any thoughts? Maybe I missed an API in there, but it looked like the framing was built directly into the decoder.
IIRC msgp allows you to unmarshal against an existing byte slice (or allocs if necessary), I think it makes use of that pretty heavily internally which might be why it's quicker in this case.
cheers
The text was updated successfully, but these errors were encountered:
First of all this is awesome! Thanks a lot for the package.
I haven't looked heavily yet but the msgp package seems to decode roughly ~70% faster given the same structure, which seems a little odd since one goal of capnproto is to reduce the allocations (and msgp has to construct everything).
I was thinking maybe splitting the framing out of the decoder so that Unmarshal can be used, allowing the user to decide whether or not it's safe to reference the memory. My main use-case for analytical type stuff where you just want to iterate and grab a field or two in-place.
Any thoughts? Maybe I missed an API in there, but it looked like the framing was built directly into the decoder.
IIRC msgp allows you to unmarshal against an existing byte slice (or allocs if necessary), I think it makes use of that pretty heavily internally which might be why it's quicker in this case.
cheers
The text was updated successfully, but these errors were encountered: