amoeba opened a new pull request, #45004:
URL: https://github.com/apache/arrow/pull/45004

   ### Rationale for this change
   
   The docstrings here are not completely accurate. See 
https://github.com/apache/arrow/issues/45003.
   
   I looked at whether it would make more sense or be more natural for methods 
like these to return an empty `dict` instead of None and I think None feels 
better from an API design and usage perspective.
   
   ### What changes are included in this PR?
   
   Updated docstrings to reflect actual behavior.
   
   ### Are these changes tested?
   
   While the changes don't need testing, I did check to see if we have unit 
tests for the behavior that's been documented here. We do.
   
   ### Are there any user-facing changes?
   
   Accurate docs.


-- 
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]

Reply via email to