Disable default value preserving (linkedin#106)

* Disable default value preserving

Author: rzhang10

core/src/main/java/org/apache/iceberg/SchemaParser.java

@@ -119,7 +119,8 @@ private static void toJson(Types.StructType struct, Integer schemaId, Set<Intege
       generator.writeBooleanField(REQUIRED, field.isRequired());
       generator.writeFieldName(TYPE);
       toJson(field.type(), generator);
-      writeDefaultValue(field.getDefaultValue(), field.type(), generator);
+      // BDP-11826: Disable serializing default value
+//      writeDefaultValue(field.getDefaultValue(), field.type(), generator);
       if (field.doc() != null) {
         generator.writeStringField(DOC, field.doc());
       }
@@ -268,13 +269,14 @@ private static Types.StructType structFromJson(JsonNode json) {
       int id = JsonUtil.getInt(ID, field);
       String name = JsonUtil.getString(NAME, field);
       Type type = typeFromJson(field.get(TYPE));
-      Object defaultValue = defaultValueFromJson(field, type);
+      // BDP-11826: Disable deserializing default value
+//      Object defaultValue = defaultValueFromJson(field, type);
       String doc = JsonUtil.getStringOrNull(DOC, field);
       boolean isRequired = JsonUtil.getBool(REQUIRED, field);
       if (isRequired) {
-        fields.add(Types.NestedField.required(id, name, type, defaultValue, doc));
+        fields.add(Types.NestedField.required(id, name, type, doc));
       } else {
-        fields.add(Types.NestedField.optional(id, name, type, defaultValue, doc));
+        fields.add(Types.NestedField.optional(id, name, type, doc));
       }
     }
 

core/src/test/java/org/apache/iceberg/TestSchemaParserForDefaultValues.java

@@ -31,6 +31,7 @@
 import org.apache.iceberg.relocated.com.google.common.collect.ImmutableMap;
 import org.apache.iceberg.types.Types.NestedField;
 import org.junit.Assert;
+import org.junit.Ignore;
 import org.junit.Test;
 
 import static org.apache.avro.Schema.Type.BOOLEAN;
@@ -42,7 +43,7 @@
 import static org.apache.avro.Schema.Type.NULL;
 import static org.apache.avro.Schema.Type.STRING;
 
-
+@Ignore("BDP-11826: Disable default value preserving in iceberg schema")
 public class TestSchemaParserForDefaultValues {
 
   private void assertEqualStructs(org.apache.iceberg.Schema expected, org.apache.iceberg.Schema actual) {