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. 👍

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.

[alec-anikin]

No chance to merge it ever. Created separate library based on this MR. https://github.com/eventso/kafka-producer

[BEagle1984]

Thank you for your work @alec-anikin.
I'm interested in taking advantage of this. @mhowlett any chance for this to be merged and maintained, or the best bet is to use the linked library?

[alec-anikin]

As I understand it, Matt no longer supports this library. There are now other maintainers here and they ignore all external mr.