Isaac7777-cpu-school opened a new pull request, #44409:
URL: https://github.com/apache/arrow/pull/44409

   ### Rationale for this change
   
   This change is to address issue #44231 in which indicates that there is 
lacking coverage of the test cases for `test_compute.py`.
   
   ### What changes are included in this PR?
   
   #### Changes according to the issue:
   
   - Added negative values for `int8`, `int16`, `int32`, `int64`, `float`, 
`double`. I have also added in negative values for test cases in `pa.list_` and 
`pa.large_list_`
   - I have added `pa.decimal128` and `pa.decimal256` in `all_array_types`.
   
   #### Further changes:
   Apart from the problems mentioned in the issue, I have also extended some of 
the tests for `pa.decimal128` and `pa.decimal256`. Currently, the 
`numerical_arrow_types` does not contain decimal. However, some of the 
functions that are tested for the `numerical_arrow_types` should also be tested 
for both decimal types. Therefore, I have duplicated similar tests but 
rewritten using decimals.
   
   Looking to close #44231 
   
   ### Are these changes tested?
   
   Yes, these changes are tests by my fork repository CI. However, as I am new 
to open source contributing, I am not sure if I have missed anything. I 
apologise if I do so.
   
   ### Are there any user-facing changes?
   
   No.


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