fix: Added an error message with column name when Series to pyarrow conver…

[karkinissan]

…sion fails. Allows for easier debugging.

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

• [✅ ] Make sure to open an issue as a [bug/issue] Arrow conversion errors do not indicate problematic dataframe column.  #1621
• [✅ ] Ensure the tests and linter pass
• [ ✅] Code coverage does not decrease (if any source code was changed)
• [ ✅] Appropriate docs were updated (if necessary)

Fixes #1621 🦕

[google-cla]

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

[conventional-commit-lint-gcf]

🤖 I detect that the PR title and the commit message differ and there's only one commit. To use the PR title for the commit history, you can use Github's automerge feature with squashing, or use automerge label. Good luck human!

-- conventional-commit-lint bot
https://conventionalcommits.org/

[chalmerlowe]

Do we have any evidence that this problem affects other conversions besides pyarrow.Array

Does a similar problem occur with:

• pyarrow.ListArray
• pyarrow.StructArray

If YES do we need to have similar error handling behavior on lines 376-379?

[karkinissan]

Good point. I would say yes. The Try block should cover lines 376 to 380.
I am not going to update the pull request for this. I have no experience creating those for large repos like this.
So please feel free to create one yourself. : )

[chalmerlowe]

The content of this PR has been incorporated in the following PR: #1836
Thanks for kicking off this effort. Much appreciated.
The other PR also includes verbiage to identify potential issues with ListArray and StructArray.
Closing this PR.

[chalmerlowe]

Closed by the merger of #1836

[karkinissan]

Thanks!