zbus-old icon indicating copy to clipboard operation
zbus-old copied to clipboard

Optimize ser/de for array of basic/fixed types

Open zeenix opened this issue 4 years ago • 3 comments
trafficstars

In GitLab by @elmarco on Mar 1, 2021, 12:55

Our Vec implementation is pretty slow, due to the genericity of value handling.

For a rough idea, it takes about 5s to ser/de a 10mb vector of bytes (ay) on a pretty fast CPU.

Here is the flamegraph link: https://elmarco.fedorapeople.org/zbus_vec_flamegraph.svg

Imho, optimizing 'ay' is the most important, as there is a higher chance to transfer big data blobs in this form.

zeenix avatar Mar 01 '21 11:03 zeenix

Imho, optimizing 'ay' is the most important, as there is a higher chance to transfer big data blobs in this form.

I agree but I've a feeling we don't need to special-case y here but implement a solution for arrays in general.

zeenix avatar Mar 01 '21 12:03 zeenix

@elmarco Wasn't this fixed with !267?

zeenix avatar Mar 14 '21 17:03 zeenix

In GitLab by @elmarco on Mar 14, 2021, 18:46

There is room for improvements for arrays others than [u8]

zeenix avatar Mar 14 '21 17:03 zeenix