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

   ### Rationale for this change
   
   As @kou suggested in 
https://github.com/apache/arrow/pull/37215#discussion_r1296559210, it makes 
sense to rename `arrow.recordbatch` (all lowercase) to `arrow.recordBatch` 
(camelCase).
   
   `recordBatch` is the first "two-word" function in the top-level `arrow.*` 
package, so it makes sense to establish a clear camelCase standard for 
"multi-word" functions in the MATLAB interface at this point.
   
   One part of the rationale motivating this change is that other language 
bindings follow a similar pattern (e.g. `pyarrow.record_batch` instead of 
`pyarrow.recordbatch`).
   
   ### What changes are included in this PR?
   
   1. Renamed `arrow.recordbatch` (all lowercase) to `arrow.recordBatch` 
(camelCase).
   2. Updated documentation and tests to reflect `arrow.recordbatch` being 
renamed to `arrow.recordBatch`.
   
   ### Are these changes tested?
   
   Yes.
   
   1. All existing MATLAB tests have been updated to use the new name 
`arrow.recordbatch` and are passing on my Debian 11 machine.
   
   ### Are there any user-facing changes?
   
   Yes.
   
   **This PR includes breaking changes to public APIs.**
   
   The public facing API `arrow.recordbatch` has been renamed to 
`arrow.recordBatch`. Since the MATLAB interface is still "pre 1.0", we don't 
expect this change to have a significant impact.
   
   ### Notes
   
   1. Thank you @sgilmore10 for your help with this pull request!


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