rdblue commented on PR #5444: URL: https://github.com/apache/iceberg/pull/5444#issuecomment-1207446722
@ebyhr, I don't think that this makes sense. `ContentFile` is the base class that we use when storing or retrieving `DataFile` and `DeleteFile` in Iceberg metadata. Adding modified time like this would require tracking file modification time in Iceberg metadata. Tracking modified time in Iceberg metadata isn't necessary. The files in an Iceberg table are considered immutable and they should never be modified. The only time that actually matters for an Iceberg file is when the file was added to Iceberg metadata, but that should be tracked by the added snapshot, not by adding more metadata to manifest files. Without a description, it's hard to tell what you're trying to accomplish with this, so I'm going to close it. If you want, add a description that explains why you're making this change and we can help you solve your problem. -- 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]
