zeroshade opened a new pull request, #517: URL: https://github.com/apache/arrow-go/pull/517
### Rationale for this change The underlying hash table is in terms of primitive types (int64/int32/int16, etc.) not the top-level types (arrow.Duration, arrow.Timestamp, etc.) which results in a panic mismatch when appending values. ### What changes are included in this PR? Checking and validating the type when appendValue is called to ensure correct type coercion. ### Are these changes tested? A unit test is added in this change. ### Are there any user-facing changes? A case that previously caused a panic will now be successful. -- 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]
