amoeba opened a new issue, #46207:
URL: https://github.com/apache/arrow/issues/46207

   ### Describe the enhancement requested
   
   [During 
review](https://github.com/apache/arrow/pull/46180#discussion_r2053064581) of 
https://github.com/apache/arrow/pull/46180, it came up that Arrow C++ has two 
`StringBuilder`s:
   
   1. The public builder for utf8() arrays, 
https://github.com/apache/arrow/blob/main/cpp/src/arrow/array/builder_binary.h#L424
 (class)
   2. A helper in 
https://github.com/apache/arrow/blob/main/cpp/src/arrow/util/string_builder.h 
(function)
   
   We should,
   
   1. Move StringBuilder (2) in cpp/src/arrow/util/string_builder.h from 
`arrow::util` to `arrow::internal` to make its intended use more clear
   2. Rename from StringBuilder to something else to avoid confusing humans and 
computers
   
   No changes would be made to the official Arrow StringBuilder class.
   
   ### Component(s)
   
   C++


-- 
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: issues-unsubscr...@arrow.apache.org.apache.org

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

Reply via email to