apache
diff --git a/‎pyiceberg/utils/schema_conversion.py‎
Lines changed: 1 addition & 2 deletions b/‎pyiceberg/utils/schema_conversion.py‎
Lines changed: 1 addition & 2 deletions
diff --git a/‎tests/integration/test_avro_compatibility.py‎
Lines changed: 0 additions & 335 deletions b/‎tests/integration/test_avro_compatibility.py‎
Lines changed: 0 additions & 335 deletions
@@ -31,7 +31,6 @@
     ICEBERG_FIELD_NAME_PROP,
     Schema,
     SchemaVisitorPerPrimitiveType,
-    _valid_avro_name,
     make_compatible_name,
     visit,
 )
@@ -541,7 +540,7 @@ def field(self, field: NestedField, field_result: AvroType) -> AvroType:
             "type": field_result if field.required else ["null", field_result],
         }
 
-        if not is_valid_field_name:
+        if orig_field_name != field_name:
             result[ICEBERG_FIELD_NAME_PROP] = orig_field_name
 
         if field.write_default is not None:
Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,6 @@`
`31`	`31`	`ICEBERG_FIELD_NAME_PROP,`
`32`	`32`	`Schema,`
`33`	`33`	`SchemaVisitorPerPrimitiveType,`
`34`		`- _valid_avro_name,`
`35`	`34`	`make_compatible_name,`
`36`	`35`	`visit,`
`37`	`36`	`)`
`@@ -541,7 +540,7 @@ def field(self, field: NestedField, field_result: AvroType) -> AvroType:`
`541`	`540`	`"type": field_result if field.required else ["null", field_result],`
`542`	`541`	`}`
`543`	`542`
`544`		`- if not is_valid_field_name:`
	`543`	`+ if orig_field_name != field_name:`
`545`	`544`	`result[ICEBERG_FIELD_NAME_PROP] = orig_field_name`
`546`	`545`
`547`	`546`	`if field.write_default is not None:`