Code review comment: convert TIME, FIXED,BINARY to VARBINARY?

Code before review:
public static Type fromIcebergType(org.apache.iceberg.types.Type icebergType) {
if (icebergType == null) {
return Type.NULL;
}
PrimitiveType primitiveType;
switch (icebergType.typeId()) {
case BOOLEAN:
primitiveType = PrimitiveType.BOOLEAN;
break;
case INTEGER:
primitiveType = PrimitiveType.INT;
break;
case LONG:
primitiveType = PrimitiveType.BIGINT;
break;
case FLOAT:
primitiveType = PrimitiveType.FLOAT;
break;
case DOUBLE:
primitiveType = PrimitiveType.DOUBLE;
break;
case DATE:
primitiveType = PrimitiveType.DATE;
break;
case TIMESTAMP:
primitiveType = PrimitiveType.DATETIME;
break;
case STRING:
case UUID:
return ScalarType.createDefaultExternalTableString();
case DECIMAL:
int precision = ((Types.DecimalType) icebergType).precision();
int scale = ((Types.DecimalType) icebergType).scale();
return ScalarType.createUnifiedDecimalType(precision, scale);
case LIST:
Type type = convertToArrayTypeForIceberg(icebergType);
if (type.isArrayType()) {
return type;
} else {
return Type.UNKNOWN_TYPE;
}
case MAP:
Type mapType = convertToMapTypeForIceberg(icebergType);
if (mapType.isMapType()) {
return mapType;
} else {
return Type.UNKNOWN_TYPE;
}
case STRUCT:
List<Types.NestedField> fields = icebergType.asStructType().fields();
Preconditions.checkArgument(fields.size() > 0);
ArrayList<StructField> structFields = new ArrayList<>(fields.size());
for (Types.NestedField field : fields) {
String fieldName = field.name();
Type fieldType = fromIcebergType(field.type());
if (fieldType.isUnknown()) {
return Type.UNKNOWN_TYPE;
}
structFields.add(new StructField(fieldName, fieldType));
}
return new StructType(structFields);
case TIME:
case FIXED:
case BINARY:
return Type.VARBINARY;
default:
primitiveType = PrimitiveType.UNKNOWN_TYPE;
}
return ScalarType.createType(primitiveType);
}

Line under review: case FIXED:

Code after addressing the review:
public static Type fromIcebergType(org.apache.iceberg.types.Type icebergType) {
if (icebergType == null) {
return Type.NULL;
}
PrimitiveType primitiveType;
switch (icebergType.typeId()) {
case BOOLEAN:
primitiveType = PrimitiveType.BOOLEAN;
break;
case INTEGER:
primitiveType = PrimitiveType.INT;
break;
case LONG:
primitiveType = PrimitiveType.BIGINT;
break;
case FLOAT:
primitiveType = PrimitiveType.FLOAT;
break;
case DOUBLE:
primitiveType = PrimitiveType.DOUBLE;
break;
case DATE:
primitiveType = PrimitiveType.DATE;
break;
case TIMESTAMP:
primitiveType = PrimitiveType.DATETIME;
break;
case STRING:
case UUID:
return ScalarType.createDefaultExternalTableString();
case DECIMAL:
int precision = ((Types.DecimalType) icebergType).precision();
int scale = ((Types.DecimalType) icebergType).scale();
return ScalarType.createUnifiedDecimalType(precision, scale);
case LIST:
Type type = convertToArrayTypeForIceberg(icebergType);
if (type.isArrayType()) {
return type;
} else {
return Type.UNKNOWN_TYPE;
}
case MAP:
Type mapType = convertToMapTypeForIceberg(icebergType);
if (mapType.isMapType()) {
return mapType;
} else {
return Type.UNKNOWN_TYPE;
}
case STRUCT:
List<Types.NestedField> fields = icebergType.asStructType().fields();
Preconditions.checkArgument(fields.size() > 0);
ArrayList<StructField> structFields = new ArrayList<>(fields.size());
for (Types.NestedField field : fields) {
String fieldName = field.name();
Type fieldType = fromIcebergType(field.type());
if (fieldType.isUnknown()) {
return Type.UNKNOWN_TYPE;
}
structFields.add(new StructField(fieldName, fieldType));
}
return new StructType(structFields);
case BINARY:
return Type.VARBINARY;
case TIME:
case FIXED:
default:
primitiveType = PrimitiveType.UNKNOWN_TYPE;
}
return ScalarType.createType(primitiveType);
}