jjerphan opened a new pull request, #35582:
URL: https://github.com/apache/arrow/pull/35582
### Rationale for this change
<!--
Why are you proposing this change? If this is already explained clearly in
the issue then this section is not needed.
Explaining clearly why changes are proposed helps reviewers understand your
changes and offer better suggestions for fixes.
-->
Addresses GH-34787.
### What changes are included in this PR?
I think this proposes the minimal viable changes to have the same signature
as proposed in GH-34787.
### Are these changes tested?
Yes, for now a single simple test has been introduced to test the correct
behavior of `ChunkedArray.to_numpy` with respect to the addition of
`zero_copy_only`.
### Are there any user-facing changes?
Yes, a minimal description of `zero_copy_only` is provided in this regard.
### Comments
Looking at the signature of `pa.Array.to_numpy`:
https://github.com/apache/arrow/blob/b211c1896916e3d0bbad1ffc88a30ddb9c90a5f0/python/pyarrow/array.pxi#L1486-L1512
I think `writable=True` can be added as a default-valued parameter to
`pa.ChunkedArray.to_numpy`.
What do you think?
--
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]
