Skip to content

Conversation

@dongxiao1198
Copy link
Contributor

1 fix TODOs in manifest adaptor
2 pass metadata to avro writer

Copy link
Member

@wgtmac wgtmac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

Thanks @dongxiao1198!

Copy link
Contributor

@gty404 gty404 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Collaborator

@zhjwpku zhjwpku left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@wgtmac wgtmac merged commit 51aa748 into apache:main Oct 17, 2025
10 checks passed
@dongxiao1198 dongxiao1198 deleted the manifest_avro_metadata branch October 20, 2025 01:41
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.

4 participants