Willem-J-an opened a new pull request, #700:
URL: https://github.com/apache/arrow-go/pull/700
### Rationale for this change
We frequently run into panic:
panic: arrow/array: column "colName" type mismatch, got=.., want=...
The got and want will be exactly the same schema, the only difference is in
the nullability of certain struct fields. The only way to figure out which
fields are the problem is to go through the debugger.
The nullability is rendered for other types like map and list, so it seems
like it should be there for struct as well.
### What changes are included in this PR?
- Add nullable in String() representation of StructType
- Corrected tests that were affected by this
- Some were just adding nullable where it's now added to the string and it
previously was not
- Some appear to be actual fixes to the tests, where the schema's did not
match, but that was not detected in the schema string comparison because
nullability was not included for structtype.
### Are these changes tested?
ran make test, didn't have the parquet test files, so didn't run those.
### Are there any user-facing changes?
Yes, the String() representation for data types is user facing.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
