Binary Producer with Span<T>

[alec-anikin]

Hi Matt, I've prepared the draft for binary producer with span api. I tried to minimize the number of changes to make it easier to review.

[ghost]

@confluentinc It looks like @alec-anikin just signed our Contributor License Agreement. :+1:

Always at your service,

clabot

[mhowlett]

thanks for the PR @alec-anikin - this will take a lot of effort to review and get merged (there's a good chance that in a PR this big we will have disagreements to work through). pretty sure I won't have time to get to it in the short term.

do you have any numbers on performance/memory improvements?

[alec-anikin]

There are no performance improvements on the library side but clients can now allocate key/values on stack (stackallock), native memory(Marshal.AllocHGlobal) and ArrayPool. So they can get a lot of improvements in terms of memory allocations.

[msallin]

Amazing work!

[msallin]

@mhowlett this should be a high priority... Any plans to work on it?

[CLAassistant]

All committers have signed the CLA.

[mhowlett]

it's on the radar, and i'd like to be working on it, but some other things have been prioritized ahead of it.