iabhi4 commented on code in PR #46815:
URL: https://github.com/apache/arrow/pull/46815#discussion_r2151131408
##########
python/pyarrow/tests/test_compute.py:
##########
@@ -1051,6 +1051,25 @@ def test_pad():
assert pc.utf8_rpad(arr, 3).tolist() == [None, 'á ', 'abcd']
+def test_utf8_zfill():
Review Comment:
Just wanted to clarify this before I made the changes! Currently, the
`utf8_zfill` kernel requires `PadOptions` (similar to how utf8_lpad behave),
and there’s no overload that accepts just `width=` as a direct keyword argument.
Would you prefer that I add a test case that intentionally omits
`PadOptions` to validate that a clear error is raised (like a `TypeError`)? Or
were you referring to a different kind of test behavior? Thanks again for the
guidance! 🙏
--
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]
