corleyma commented on issue #1571: URL: https://github.com/apache/iceberg-python/issues/1571#issuecomment-2616783358
I think the problem here (and in other mentions I've seen of this) is that folks are attempting to create row-level overwrite filters, which is not what this API is really for. We can/should probably fix the recursion error (by changing the code not to be recursive, probably), but it still seems like a smell that people are thinking about things incorrectly. Ultimately the filter conditions should identify which partitions/data files need changes, not which rows. Using this as a crude replacement for Merge Into requires you to understand your data layout well and how iceberg works in general, so I don't think we should be advising it in the general case. -- 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
