[ARROW-7084] [C++] ArrayRangeEquals should check for full type equality?

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.0.0
• Component/s: C++
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/23391
• Assignee: Antoine Pitrou
• Reporter: Micah Kornfield
• Created: 07/Nov/19 09:06
• Updated: 11/Jan/23 07:51
• Resolved: 22/Jun/20 15:39

Description

It looks like ArrayRangeEquals in compare.cc only checks type IDs before doing comparison actual values.  This is inconsistent with ArrayEquals which checks for type equality and also seems incorrect for cases like Decimal128. 

 

I presume this was an oversight when fixing ARROW-2567 but maybe it was intentional?

uwe?

Issue Links

• links to: GitHub Pull Request #7496