sungwy opened a new pull request, #965: URL: https://github.com/apache/iceberg-python/pull/965
Fixes: https://github.com/apache/iceberg-python/issues/953 This may be a backward incompatible change because it seemingly removes support for allowing the table name within the parser string expression. However, a `row_filter` is always specified on a table Scan, and hence supporting parsing of the table name on a `row_filter` feels redundant. This assumption, also prevents us from being able to parse a nested field in the string expression, as we will always only take the name of the field at the lowest level, which prevents us from binding the field using the name in the expression string. NOTE: There is a larger conversation around correctly supporting name mapping, and the correctness of expressing field names in flat form https://github.com/apache/iceberg-python/issues/935 -- 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
