fix build failure

Author: gty404

src/iceberg/test/update_schema_test.cc

@@ -1478,7 +1478,8 @@ TEST_F(UpdateSchemaTest, TestMultipleMoves) {
   EXPECT_EQ(fields[0].name(), "z");
   EXPECT_EQ(fields[1].name(), "y");
 
-  int w_pos = -1, x_pos = -1;
+  int w_pos = -1;
+  int x_pos = -1;
   for (size_t i = 0; i < fields.size(); ++i) {
     if (fields[i].name() == "w") w_pos = i;
     if (fields[i].name() == "x") x_pos = i;
@@ -1507,7 +1508,8 @@ TEST_F(UpdateSchemaTest, TestMoveTopLevelColumnBeforeFirst) {
   ASSERT_TRUE(result.schema != nullptr);
 
   const auto& fields = result.schema->fields();
-  int y_pos = -1, x_pos = -1;
+  int y_pos = -1;
+  int x_pos = -1;
   for (size_t i = 0; i < fields.size(); ++i) {
     if (fields[i].name() == "y") y_pos = i;
     if (fields[i].name() == "x") x_pos = i;
@@ -1541,7 +1543,8 @@ TEST_F(UpdateSchemaTest, TestMoveTopLevelColumnAfter) {
   ASSERT_TRUE(result.schema != nullptr);
 
   const auto& fields = result.schema->fields();
-  int w_pos = -1, x_pos = -1;
+  int w_pos = -1;
+  int x_pos = -1;
   for (size_t i = 0; i < fields.size(); ++i) {
     if (fields[i].name() == "w") w_pos = i;
     if (fields[i].name() == "x") x_pos = i;

src/iceberg/type.cc

@@ -156,7 +156,7 @@ TypeId ListType::type_id() const { return kTypeId; }
 const SchemaField& ListType::element() const { return element_; }
 
 std::string ListType::ToString() const {
-  return std::format("list<{}>", element_.type()->ToString());
+  return std::vformat("list<{}>", std::make_format_args(element_));
 }
 
 std::span<const SchemaField> ListType::fields() const { return {&element_, 1}; }
@@ -213,7 +213,7 @@ const SchemaField& MapType::value() const { return fields_[1]; }
 TypeId MapType::type_id() const { return kTypeId; }
 
 std::string MapType::ToString() const {
-  return std::format("map<{}, {}>", key().type()->ToString(), value().type()->ToString());
+  return std::vformat("map<{}: {}>", std::make_format_args(key(), value()));
 }
 
 std::span<const SchemaField> MapType::fields() const { return fields_; }

src/iceberg/update/update_schema.cc

@@ -752,8 +752,7 @@ UpdateSchema& UpdateSchema::MoveInternal(std::string_view name, const Move& move
 
     const auto& parent_field = parent_field_result.value()->get();
     ICEBERG_BUILDER_CHECK(parent_field.type()->type_id() == TypeId::kStruct,
-                          "Cannot move fields in non-struct type: {}",
-                          parent_field.type()->ToString());
+                          "Cannot move fields in non-struct type");
 
     if (move.type == Move::MoveType::kBefore || move.type == Move::MoveType::kAfter) {
       auto ref_parent_it = id_to_parent_.find(move.reference_field_id);