[ARROW-15166] [C++] Implement an array_filter kernel for decimal256

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 7.0.0
• Component/s: C++
• Labels: good-first-issue kernel pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/30670
• Assignee: David Li
• Reporter: Dragoș Moldovan-Grünfeld
• Created: 20/Dec/21 16:31
• Updated: 11/Jan/23 08:44
• Resolved: 29/Dec/21 12:30

Description

This surfaced in some of the R unit tests. I believe the expected behaviour would be to return an empty chunked array. Reproducible example:

suppressMessages(library(arrow))

empty_filter <- ChunkedArray$create(type = bool())
one_empty_chunk <- ChunkedArray$create(type = decimal128(4, 2))
one_empty_chunk$Filter(empty_filter)
#> ChunkedArray
#> [
#> 
#> ]

one_empty_chunk <- ChunkedArray$create(type = decimal256(4, 2))
one_empty_chunk$Filter(empty_filter)
#>Error: NotImplemented: Function array_filter has no kernel matching input types (array[decimal256(4, 2)], array[bool])

This suite of unit tests were introduced as part of ARROW-13761.

Issue Links

• is depended upon by: ARROW-15387 [R] Un-skip chunked array test for decimal256()
• is related to: ARROW-13761 [R] arrow::filter() crashes (aborts R session)
• links to: GitHub Pull Request #12047