jorisvandenbossche commented on a change in pull request #8044:
URL: https://github.com/apache/arrow/pull/8044#discussion_r478879571
##########
File path: python/pyarrow/tests/test_compute.py
##########
@@ -562,7 +562,7 @@ def test_is_null():
def test_fill_null():
arr = pa.array([1, 2, None, 4], type=pa.int8())
fill_value = pa.array([5], type=pa.int8())
- with pytest.raises(TypeError):
+ with pytest.raises(pa.ArrowInvalid):
Review comment:
Hmm, for this case I find the original error message clearer ..
That's the consequence of the `scalar(..)` conversion using the array
conversion under the hood, I suppose?
But OK, I suppose this is fine (it's maybe mainly the multiline repr of the
array in the middle of the sentence that makes it more confusing)
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
