Skip to content

Conversation

@yongtang
Copy link
Member

Signed-off-by: Yong Tang yong.tang.github@outlook.com

Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
@yongtang yongtang marked this pull request as ready for review August 10, 2022 13:28
Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
@yongtang
Copy link
Member Author

@kvignesh1420 The PR has been updated to include an additional test case. Please take a look.

@kvignesh1420
Copy link
Member

@yongtang LGTM, let's see if the tests pass this time. The lint test fix might need a separate PR.

@yongtang
Copy link
Member Author

@kvignesh1420 Thanks. Will merge in for now and will create a separate PR for follow up the other fix.

@yongtang yongtang merged commit 24aae0a into tensorflow:master Aug 16, 2022
@yongtang yongtang deleted the arrow branch August 16, 2022 19:10
zheolong pushed a commit to zheolong/io-1 that referenced this pull request Jul 24, 2025
* Add arrow binary data type support

Signed-off-by: Yong Tang <yong.tang.github@outlook.com>

* Add test case for string type

Signed-off-by: Yong Tang <yong.tang.github@outlook.com>

Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
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.

2 participants