Skip to content

Add Avro file for null test case - #90

Merged
alamb merged 2 commits into
apache:masterfrom
nenorbot:master
May 10, 2023
Merged

alamb merged 2 commits into
apache:masterfrom
nenorbot:master

Conversation

@nenorbot

@nenorbot nenorbot commented May 10, 2023 •

Copy link
Copy Markdown
Contributor

To be used for the changes in apache/datafusion#6307

file contents:

❯ avro-tools tojson data/avro/alltypes_nulls_plain.avro
{"string_col":null,"int_col":null,"bool_col":null,"bigint_col":null,"float_col":null,"double_col":null,"bytes_col":null}

@alamb alamb left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks @nenorbot -- looks good to me and I think a 603 byte file is perfect

I wonder if you could update the https://github.com/apache/arrow-testing/tree/master/data/avro#readme readme file as well with the information about this file (perhaps the PR description)

@nenorbot

Copy link
Copy Markdown
Contributor Author

I wonder if you could update the https://github.com/apache/arrow-testing/tree/master/data/avro#readme readme file as well with the information about this file (perhaps the PR description)

@alamb Done, added a note about the new file

@alamb alamb left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you!

@alamb
alamb merged commit e81d0c6 into apache:master May 10, 2023
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