Awesome06 opened a new pull request, #55257:
URL: https://github.com/apache/arrow/pull/55257

   ### Rationale for this change
   
   Vector replace functions (`replace_with_mask`, `fill_null_forward`, and 
`fill_null_backward`) raised `NotImplemented` for `string_view`, `binary_view`, 
`decimal32`, and `decimal64` because kernels were missing registrations for 
these types in `vector_replace.cc`.
   
   ### What changes are included in this PR?
   
   - Registered `Type::DECIMAL32` and `Type::DECIMAL64` kernels using 
`Functor<FixedSizeBinaryType>`.
   - Implemented `ReplaceMaskImpl` and `FillNullImpl` specializations for 
binary view types using `BinaryViewBuilder`.
   - Registered `Type::BINARY_VIEW` and `Type::STRING_VIEW` kernels across 
`replace_with_mask`, `fill_null_forward`, and `fill_null_backward`.
   - Added unit tests in `vector_replace_test.cc` covering inline vs 
out-of-line storage tiers, asymmetric slicing, embedded null characters, and 
decimal precision/scale validation.
   - Added Python unit tests in `test_compute.py`.
   
   ### Are these changes tested?
   
   Yes. Verified with unit tests in `vector_replace_test.cc` and 
`python/pyarrow/tests/test_compute.py`.
   
   ### Are there any user-facing changes?
   
   No breaking changes. Extends existing vector replace functions to support 
view and small decimal types.
   
   Closes #55256
   * GitHub Issue: #55256


-- 
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]

Reply via email to