pvary commented on code in PR #14414:
URL: https://github.com/apache/iceberg/pull/14414#discussion_r2509997695
##########
core/src/main/java/org/apache/iceberg/ManifestReader.java:
##########
@@ -145,6 +147,18 @@ private <T extends ContentFile<T>> PartitionSpec
readPartitionSpec(InputFile inp
return PartitionSpecParser.fromJsonFields(schema, specId,
metadata.get("partition-spec"));
}
+ private int readFormatVersion(InputFile inputFile) {
+ Map<String, String> metadata = readMetadata(inputFile);
Review Comment:
When we were working on the File Format API, there was a discussion around
providing API for accessing file metadata. The consensus was that the metadata
should only be used for debug purposes.
Do we have other ways to decide the version of a manifest file?
--
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]