redpanda icon indicating copy to clipboard operation
redpanda copied to clipboard

Published 20 hours ago • verified publisher icon redpanda-data

serde: add serde_direct_read

Open felixguendling opened this issue 1 year ago • 0 comments

Cover letter

Describe in plain language the motivation (bug, feature, etc.) behind the change in this PR and how the included commits address it.

Fixes #ISSUE-NUMBER, Fixes #ISSUE-NUMBER, ...

Backport Required

UX changes

Describe in plain language how this PR affects an end-user. What topic flags, configuration flags, command line flags, deprecation policies etc are added/changed.

Release notes

felixguendling avatar Aug 08 '22 12:08 felixguendling

Should I pull your changes into my branch to add them to the PR or do you want to create a separate one with my and your changes? Alternatively, we can also merge this first and then create another PR with your changes.

felixguendling avatar Aug 18 '22 10:08 felixguendling

If you think df730cd7a01f67b3d410edc119baf24054859ba1 makes sense, can you grab it into this PR. The effort to remove the default constructors can happen separately.

BenPope avatar Aug 18 '22 10:08 BenPope

Yes, looks good :+1:

felixguendling avatar Aug 19 '22 06:08 felixguendling