1

Author: HeartLinked

src/iceberg/catalog/rest/validator.cc

@@ -24,6 +24,7 @@
 
 #include "iceberg/catalog/rest/types.h"
 #include "iceberg/result.h"
+#include "iceberg/util/formatter_internal.h"
 #include "iceberg/util/macros.h"
 
 namespace iceberg::rest {
@@ -91,16 +92,9 @@ Status Validator::Validate(const UpdateNamespacePropertiesRequest& request) {
                                 std::back_inserter(common));
 
   if (!common.empty()) {
-    std::string keys;
-    for (size_t i = 0; i < common.size(); ++i) {
-      if (i) keys += ", ";
-      keys += common[i];
-    }
-
     return Invalid(
-        "Invalid namespace update: cannot simultaneously set and remove keys: "
-        "[{}]",
-        keys);
+        "Invalid namespace update: cannot simultaneously set and remove keys: {}",
+        common);
   }
   return {};
 }