[GitHub] [arrow] hermanschaaf opened a new issue, #35761: [Go] TypeEqual returns false for equivalent maps and true for non-equivalent maps

["hermanschaaf (via GitHub)" <gi...@apache.org>]

hermanschaaf opened a new issue, #35761:
URL: https://github.com/apache/arrow/issues/35761

   ### Describe the bug, including details regarding any error messages, version, and platform.
   
   Currently the implementation of TypeEqual for maps falls back to `reflect.DeepEqual`. However, this falls short in a couple of cases:
    - it doesn't use TypeEqual to compare the underlying key/item types, so types that would normally be considered equal fail the equivalence test when used in a map (e.g. timestamps with UTC location vs nil location usually return as equal, but not when placed in a map)
    - the passed in metadata options are ignored, so even when TypeEqual is instructed not to consider metadata, keys or items with differing metadata will cause the function to return false
   
   I will open a PR to address these issues.
   
   ### Component(s)
   
   Go


-- 
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: issues-unsubscribe@arrow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [arrow] zeroshade closed issue #35761: [Go] TypeEqual returns false for equivalent maps and true for non-equivalent maps

["zeroshade (via GitHub)" <gi...@apache.org>]

zeroshade closed issue #35761: [Go] TypeEqual returns false for equivalent maps and true for non-equivalent maps
URL: https://github.com/apache/arrow/issues/35761


-- 
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: issues-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org