rdblue commented on a change in pull request #1219:
URL: https://github.com/apache/iceberg/pull/1219#discussion_r466020016
##########
File path: core/src/main/java/org/apache/iceberg/V2Metadata.java
##########
@@ -56,87 +58,31 @@ private V2Metadata() {
* This is used to maintain compatibility with v2 by writing manifest list
files with the old schema, instead of
* writing a sequence number into metadata files in v2 tables.
*/
- static class IndexedManifestFile implements ManifestFile, IndexedRecord {
+ static class IndexedManifestFile implements ManifestFile, Record {
Review comment:
The Indexed* classes can still be used for writing. That way, we can
convert the read path and then convert the write path later.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
