Fokko opened a new pull request, #6141:
URL: https://github.com/apache/iceberg/pull/6141
Currently, we silently turn Literals into None if we can't convert them,
instead I prefer to raise an exception. This can cause silent bugs like:
`EqualTo(Reference("id"), StringLiteral("123a"))` will turn into a `IsNull`
predicate since `123a` cannot be casted to a `Long` (assuming that the `id`
column is a Long). This PR also adds support of casting a `StringLiteral` to a
`LongLiteral`.
Blocked on https://github.com/apache/iceberg/pull/6140
--
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
