mr-smidge opened a new pull request #7671:
URL: https://github.com/apache/arrow/pull/7671


   This PR fixes a few bugs in `BinaryArray.Builder()`:
   * Fixes the `Clear()` method, which previously would break all 
subsequently-appended values (see JIRA ticket for examples).
   * Makes the `Build()` method idempotent, by making sure the builder's 
internal arrays are not modified when it is called.
   
   And makes a few other improvements:
   * Comprehensive test coverage of the builder's key methods (this was written 
in proper TDD-style, so these tests would fail if run on `master`).
   * XML docstrings for all methods affected.


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

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to