[Python] Improve error message in compute functions when passing wrong positional/keyword arguments

[asfimport]

From https://stackoverflow.com/questions/69990087/how-to-use-pyarrow-compute-is-in-function

Small example of the current error messages:

array1 = pa.array([1,2,3,4])
array2 = pa.array([1,4])

import pyarrow.compute as pc

# Using positional for value_set as well (the signature indicates with * that this are keyword
# only arguments, but the error message is still confusing compared to plain python
>>> pc.is_in(array1, array2)
...
ArrowInvalid: Function is_in accepts 1 arguments but attempted to look up kernel(s) with 2

# Using keyword for the first argument (which is allowed in pure python functions)
# gives confusing error message because it passes it to the options
>>> pc.is_in(values=array1,value_set=array2)
...
~/scipy/repos/arrow/python/pyarrow/_compute.pyx in pyarrow._compute.SetLookupOptions.__init__()

TypeError: __init__() got an unexpected keyword argument 'values'

I think it should be quite easy to improve the error messaging on this front, since we know how many arguments the kernel takes.

cc @amol-

Reporter: Joris Van den Bossche / @jorisvandenbossche
Assignee: Joris Van den Bossche / @jorisvandenbossche

PRs and other links:

• GitHub Pull Request #11721

_{Note: This issue was originally created as ARROW-14732. Please see the migration documentation for further details.}

[asfimport]

Antoine Pitrou / @pitrou:
Issue resolved by pull request 11721
#11721