manirajv06 commented on PR #12611: URL: https://github.com/apache/iceberg/pull/12611#issuecomment-2745114645
@aihuaxu Since this is first PR, need to discuss few things before in hand to ensure the direction is correct. Hence, marking the pr as draft for now. Added a new method to sanitize the variant fields only for `BoundPredicate` expression code path. Need to do the same for `UnboundPredicate` as well. It requires us to add a new class `VariantLiteral` similar to any other `Literal` for other types like int, float etc. Is this correct? Can you confirm this? Also, need to add the test as well. -- 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
