JFinis commented on issue #8684: URL: https://github.com/apache/iceberg/issues/8684#issuecomment-1742722121
@Fokko What is the intended future of your PR? If I read the discussion in there correctly, then the spec is correct, as the same field is also used to track delete files, so `data_` is fully incorrect here and thus instead the field names should be adapted to the spec. Do I get the gist of the discussion correctly? If so, do I understand correctly that we all agree that this should be fixed in the implementation, it was just not merged, yet? (I'm trying to gauge in which direction this will go, so I can adapt my implementation to either also write `data_` or not.) -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
