Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add AppendBinary to append avro data without getting binary from native #251

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

CharmCcc
Copy link

AppendBinary appends one or more avro data from pulsar/kafka to an OCF file in a block.

AppendBinary  appends one or more avro data from pulsar/kafka  to an OCF file in a block.
@CharmCcc CharmCcc changed the title Update ocf_writer.go Add AppendBinary to append avro data without getting binary from native Jul 12, 2022
@CharmCcc
Copy link
Author

Now I have a lot of avro data from pulsar/kafka, I want to use ocfwrite to append them to an OCF file.

@xmcqueen
Copy link
Contributor

xmcqueen commented Aug 19, 2022

Hi, and thanks! I don't see anything pulsar/kafka specific in the PR. It looks like a general utility for the OCF functionality and seems desirable. Is that right?

Would you be willing to provide some unit tests? I'd feel more confident about a merge if we had some passing tests in place for this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants