GH-46207: [C++] Rename arrow::util::StringBuilder and move to internal namespace (#46813)

### Rationale for this change

- Move and Rename function `arrow::util::StringBuilder` to internal namespace to avoid confusion with class `arrow::StringBuilder`

### What changes are included in this PR?

- Move `arrow::util::StringBuilder` to `arrow::internal::StringBuilder`
- Rename `arrow::internal::StringBuilder` to `arrow::internal::JoinToString`
- Rename `arrow/util/string_builder.{h|cc}` to `arrow/util/string_util_internal.{h|cc}` 

### Are these changes tested?

Yes.

### Are there any user-facing changes?

No. they are used internally.

* GitHub Issue: #46207

Authored-by: Ziy1-Tan <[email protected]>
Signed-off-by: Antoine Pitrou <[email protected]>

Author: Ziy1-Tan

cpp/src/arrow/CMakeLists.txt

@@ -517,7 +517,7 @@ set(ARROW_UTIL_SRCS
     util/ree_util.cc
     util/secure_string.cc
     util/string.cc
-    util/string_builder.cc
+    util/string_util.cc
     util/task_group.cc
     util/tdigest.cc
     util/thread_pool.cc

cpp/src/arrow/flight/transport/grpc/util_internal.cc

@@ -29,7 +29,7 @@
 #include "arrow/flight/types.h"
 #include "arrow/status.h"
 #include "arrow/util/string.h"
-#include "arrow/util/string_builder.h"
+#include "arrow/util/string_util.h"
 
 namespace arrow {
 
@@ -134,9 +134,10 @@ static TransportStatus TransportStatusFromGrpc(const ::grpc::Status& grpc_status
       return TransportStatus{TransportStatusCode::kUnauthenticated,
                              grpc_status.error_message()};
     default:
-      return TransportStatus{TransportStatusCode::kUnknown,
-                             util::StringBuilder("(", grpc_status.error_code(), ")",
-                                                 grpc_status.error_message())};
+      return TransportStatus{
+          TransportStatusCode::kUnknown,
+          arrow::internal::JoinToString("(", grpc_status.error_code(), ")",
+                                        grpc_status.error_message())};
   }
 }
 

cpp/src/arrow/flight/types.cc

@@ -37,7 +37,7 @@
 #include "arrow/util/formatting.h"
 #include "arrow/util/logging.h"
 #include "arrow/util/string.h"
-#include "arrow/util/string_builder.h"
+#include "arrow/util/string_util.h"
 #include "arrow/util/uri.h"
 
 namespace arrow {
@@ -189,8 +189,8 @@ static std::ostream& operator<<(std::ostream& os, std::map<std::string, T> m) {
 // Wrapper types for Flight RPC protobuf messages
 
 std::string BasicAuth::ToString() const {
-  return arrow::util::StringBuilder("<BasicAuth username='", username,
-                                    "' password=(redacted)>");
+  return arrow::internal::JoinToString("<BasicAuth username='", username,
+                                       "' password=(redacted)>");
 }
 
 bool BasicAuth::Equals(const BasicAuth& other) const {
@@ -886,8 +886,8 @@ Status FlightPayload::Validate() const {
 }
 
 std::string ActionType::ToString() const {
-  return arrow::util::StringBuilder("<ActionType type='", type, "' description='",
-                                    description, "'>");
+  return arrow::internal::JoinToString("<ActionType type='", type, "' description='",
+                                       description, "'>");
 }
 
 const ActionType ActionType::kCancelFlightInfo =
@@ -930,7 +930,7 @@ arrow::Status ActionType::Deserialize(std::string_view serialized, ActionType* o
 }
 
 std::string Criteria::ToString() const {
-  return arrow::util::StringBuilder("<Criteria expression='", expression, "'>");
+  return arrow::internal::JoinToString("<Criteria expression='", expression, "'>");
 }
 
 bool Criteria::Equals(const Criteria& other) const {

cpp/src/arrow/meson.build

@@ -199,7 +199,7 @@ arrow_util_srcs = [
     'util/mutex.cc',
     'util/ree_util.cc',
     'util/string.cc',
-    'util/string_builder.cc',
+    'util/string_util.cc',
     'util/task_group.cc',
     'util/tdigest.cc',
     'util/thread_pool.cc',

cpp/src/arrow/status.h

@@ -23,7 +23,7 @@
 
 #include "arrow/util/compare.h"
 #include "arrow/util/macros.h"
-#include "arrow/util/string_builder.h"
+#include "arrow/util/string_util.h"
 #include "arrow/util/visibility.h"
 
 #ifdef ARROW_EXTRA_ERROR_CONTEXT
@@ -181,13 +181,13 @@ class ARROW_EXPORT [[nodiscard]] Status : public util::EqualityComparable<Status
 
   template <typename... Args>
   static Status FromArgs(StatusCode code, Args&&... args) {
-    return Status(code, util::StringBuilder(std::forward<Args>(args)...));
+    return Status(code, internal::JoinToString(std::forward<Args>(args)...));
   }
 
   template <typename... Args>
   static Status FromDetailAndArgs(StatusCode code, std::shared_ptr<StatusDetail> detail,
                                   Args&&... args) {
-    return Status(code, util::StringBuilder(std::forward<Args>(args)...),
+    return Status(code, internal::JoinToString(std::forward<Args>(args)...),
                   std::move(detail));
   }
 

cpp/src/arrow/testing/gtest_util.h

@@ -40,7 +40,7 @@
 #include "arrow/type_fwd.h"
 #include "arrow/type_traits.h"
 #include "arrow/util/macros.h"
-#include "arrow/util/string_builder.h"
+#include "arrow/util/string_util.h"
 #include "arrow/util/type_fwd.h"
 
 // NOTE: failing must be inline in the macros below, to get correct file / line number
@@ -142,7 +142,7 @@
 // A generalized version of GTest's SCOPED_TRACE that takes arbitrary arguments.
 //   ARROW_SCOPED_TRACE("some variable = ", some_variable, ...)
 
-#define ARROW_SCOPED_TRACE(...) SCOPED_TRACE(::arrow::util::StringBuilder(__VA_ARGS__))
+#define ARROW_SCOPED_TRACE(...) SCOPED_TRACE(::arrow::internal::JoinToString(__VA_ARGS__))
 
 namespace arrow {
 

cpp/src/arrow/util/bitmap.h

@@ -37,7 +37,7 @@
 #include "arrow/util/endian.h"
 #include "arrow/util/functional.h"
 #include "arrow/util/span.h"
-#include "arrow/util/string_builder.h"
+#include "arrow/util/string_util.h"
 #include "arrow/util/visibility.h"
 
 namespace arrow {

cpp/src/arrow/util/bitset_stack_internal.h

@@ -38,7 +38,7 @@
 #include "arrow/util/compare.h"
 #include "arrow/util/functional.h"
 #include "arrow/util/macros.h"
-#include "arrow/util/string_builder.h"
+#include "arrow/util/string_util.h"
 #include "arrow/util/type_traits.h"
 #include "arrow/util/visibility.h"
 

cpp/src/arrow/util/meson.build

@@ -160,7 +160,7 @@ install_headers(
         'simd.h',
         'small_vector.h',
         'span.h',
-        'string_builder.h',
+        'string_util.h',
         'string.h',
         'task_group.h',
         'test_common.h',

cpp/src/arrow/util/string_builder.cc renamed to cpp/src/arrow/util/string_util.cc

@@ -15,15 +15,13 @@
 // specific language governing permissions and limitations
 // under the License.
 
-#include "arrow/util/string_builder.h"
+#include "arrow/util/string_util.h"
 
 #include <memory>
 #include <sstream>
 
 namespace arrow {
-
-namespace util {
-namespace detail {
+namespace internal {
 
 StringStreamWrapper::StringStreamWrapper()
     : sstream_(std::make_unique<std::ostringstream>()), ostream_(*sstream_) {}
@@ -32,6 +30,5 @@ StringStreamWrapper::~StringStreamWrapper() {}
 
 std::string StringStreamWrapper::str() { return sstream_->str(); }
 
-}  // namespace detail
-}  // namespace util
+}  // namespace internal
 }  // namespace arrow