rdblue commented on PR #8678: URL: https://github.com/apache/iceberg/pull/8678#issuecomment-1758155921
I think we may want to consider an alternative, which is to get rid of `operation` entirely. What is the value of `operation`? In a table, we use the operation to make assumptions about metadata without actually reading parts of the metadata tree. For views, I don't think there is a similar need -- we probably won't use this anywhere. If we don't need operation to make something faster, then why include it at all? It just creates an awkward situation here where we need special logic to deduplicate a version. Getting rid of it and checking the whole summary makes more sense to me. -- 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
