https://pantsbuild.org/ logo
#announce
Title
# announce
f

fast-nail-55400

12/18/2020, 10:30 PM
I do think we should be consuming Bytes support for Prost protobuf types so we can avoid one of the double copies in the remote store code
I’m working on that presently but overriding transitive deps in cargo is not quite working right for me
(the Bytes support is only on Prost
master
)
a

average-vr-56795

12/18/2020, 11:22 PM
Let me know if you want a second pair of eyes - happy to look at a branch :)
f

fast-nail-55400

12/18/2020, 11:25 PM
I figured it out. Just posted PR. I solved the issue by not consuming past the Bytes 0.6 upgrade in Prost.
w

witty-crayon-22786

12/21/2020, 8:57 PM
ftr, my understanding is that using
Bytes
avoids a lot of double copies. because many of the fields of a message can be
Bytes
“pointers”/views of the original bytes of the message when they come off the wire