This is an automated email from the ASF dual-hosted git repository.

fokko pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/iceberg.git


The following commit(s) were added to refs/heads/master by this push:
     new f688363a9a Core: FixupTypes copied doc info (#8416)
f688363a9a is described below

commit f688363a9aad31d24f60fee4ce6952d6d991bc92
Author: baiyangtx <[email protected]>
AuthorDate: Thu Aug 31 14:48:57 2023 +0800

    Core: FixupTypes copied doc info (#8416)
    
    * Fix: FixupTypes lost doc info
    
    * Run spotlessApply
---
 core/src/main/java/org/apache/iceberg/types/FixupTypes.java | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/core/src/main/java/org/apache/iceberg/types/FixupTypes.java 
b/core/src/main/java/org/apache/iceberg/types/FixupTypes.java
index 7ae36e4884..23fccddda3 100644
--- a/core/src/main/java/org/apache/iceberg/types/FixupTypes.java
+++ b/core/src/main/java/org/apache/iceberg/types/FixupTypes.java
@@ -62,11 +62,13 @@ public abstract class FixupTypes extends 
TypeUtil.CustomOrderSchemaVisitor<Type>
 
       } else if (field.isRequired()) {
         hasChange = true;
-        newFields.add(Types.NestedField.required(field.fieldId(), 
field.name(), resultType));
+        newFields.add(
+            Types.NestedField.required(field.fieldId(), field.name(), 
resultType, field.doc()));
 
       } else {
         hasChange = true;
-        newFields.add(Types.NestedField.optional(field.fieldId(), 
field.name(), resultType));
+        newFields.add(
+            Types.NestedField.optional(field.fieldId(), field.name(), 
resultType, field.doc()));
       }
     }
 

Reply via email to