stevenzwu opened a new pull request, #13895:
URL: https://github.com/apache/iceberg/pull/13895
In the field, we have seen some writer produce non-conform metadata in
Manifest Avro file.
```
partition-spec: {"spec-id":0,"fields":[]}
```
Probably the spec wording caused the mis-interpretation that it is the JSON
serialization of the whole partition spec. But the spec and the Java reference
implementation meant only the partition fields array of the partition spec, as
the spec id was encoded as a separate metadata field in the Avro file.
```
.meta("partition-spec", PartitionSpecParser.toJsonFields(spec))
.meta("partition-spec-id", String.valueOf(spec.specId()))
```
This PR is to clarify this Avro metadata field.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
