format code

Author: MisterRaindrop

src/iceberg/avro/avro_schema_util.cc

@@ -240,7 +240,8 @@ Status ToAvroNodeVisitor::Visit(const StructType& type, ::avro::NodePtr* node) {
     ICEBERG_RETURN_UNEXPECTED(Visit(sub_field, &field_node));
 
     bool isValidFieldName = validAvroName(std::string(sub_field.name()));
-    std::string fieldName = isValidFieldName ? std::string(sub_field.name()) : SanitizeFieldName(sub_field.name());
+    std::string fieldName = isValidFieldName ? std::string(sub_field.name())
+                                             : SanitizeFieldName(sub_field.name());
     (*node)->addName(fieldName);
     (*node)->addLeaf(field_node);
     (*node)->addCustomAttributesForField(GetAttributesWithFieldId(sub_field.field_id()));

src/iceberg/avro/avro_schema_util_internal.h

@@ -169,11 +169,13 @@ Result<::avro::NodePtr> MakeAvroNodeWithFieldIds(const ::avro::NodePtr& original
 /// Conversion rules:
 /// 1. If the first character is not a letter or underscore, it is specially handled:
 ///    - Digits: Prefixed with an underscore (e.g., '3' -> '_3')
-///    - Other characters: Converted to '_x' followed by the uppercase hexadecimal representation
+///    - Other characters: Converted to '_x' followed by the uppercase hexadecimal
+///    representation
 ///      of the character (e.g., '$' -> '_x24')
 /// 2. For characters other than the first:
 ///    - If it's a letter, digit, or underscore, it remains unchanged
-///    - Other characters: Converted to '_x' followed by the uppercase hexadecimal representation
+///    - Other characters: Converted to '_x' followed by the uppercase hexadecimal
+///    representation
 ///
 /// Examples:
 /// - "123field" -> "_123field"

test/avro_schema_test.cc

@@ -73,7 +73,7 @@ TEST(ValidAvroNameTest, InvalidNames) {
   // Names starting with numbers should return false
   EXPECT_FALSE(validAvroName("123field"));
   EXPECT_FALSE(validAvroName("0value"));
-  
+
   // Names with special characters should return false
   EXPECT_FALSE(validAvroName("field-name"));
   EXPECT_FALSE(validAvroName("field.name"));
@@ -109,7 +109,8 @@ TEST(SanitizeFieldNameTest, InvalidFieldNames) {
   EXPECT_EQ(SanitizeFieldName("field#name"), "field_x23name");
 
   // Complex field names with multiple issues
-  EXPECT_EQ(SanitizeFieldName("1field-with.special@chars"), "_1field_x2Dwith_x2Especial_x40chars");
+  EXPECT_EQ(SanitizeFieldName("1field-with.special@chars"),
+            "_1field_x2Dwith_x2Especial_x40chars");
   EXPECT_EQ(SanitizeFieldName("user-email"), "user_x2Demail");
 }
 
@@ -275,11 +276,13 @@ TEST(ToAvroNodeVisitorTest, StructTypeWithSanitizedFieldNames) {
 
   // Check that field names are sanitized
   ASSERT_EQ(node->names(), 4);
-  EXPECT_EQ(node->nameAt(0), "user_x2Dname");      // "user-name" -> "user_x2Dname"
-  EXPECT_EQ(node->nameAt(1), "email_x2Eaddress");  // "email.address" -> "email_x2Eaddress"
-  EXPECT_EQ(node->nameAt(2), "_123field");      // "123field" -> "_123field"
-  EXPECT_EQ(node->nameAt(3),
-            "field_x20with_x20spaces");  // "field with spaces" -> "field_x20with_x20spaces"
+  EXPECT_EQ(node->nameAt(0), "user_x2Dname");   // "user-name" -> "user_x2Dname"
+  EXPECT_EQ(node->nameAt(1),
+            "email_x2Eaddress");            // "email.address" -> "email_x2Eaddress"
+  EXPECT_EQ(node->nameAt(2), "_123field");  // "123field" -> "_123field"
+  EXPECT_EQ(
+      node->nameAt(3),
+      "field_x20with_x20spaces");  // "field with spaces" -> "field_x20with_x20spaces"
 
   // Check that field IDs are correctly applied
   // Each field has 1 custom attribute: field-id