manirajv06 commented on PR #12611: URL: https://github.com/apache/iceberg/pull/12611#issuecomment-2751861783
@RussellSpitzer Thanks for the quick reply. We do have literal for every data type being supported. Not only this, there are two variants of `sanitize()` to handle more or less same data types, with one being responsible to handle `Literal` based argument. So, I thought we might to do the same even for Variant type as well. Please help me to understand if i am wrong. If not required, then are we good with `sanitizeVariant(VariantValue value, long now, int today)` itself? Can I go ahead and add test for the same? -- 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
