manirajv06 commented on PR #13137: URL: https://github.com/apache/iceberg/pull/13137#issuecomment-3001234585
Just to make it clear, I haven't used comparators in VariantLiteral as each element would be extracted using "extract" construct in actual usage. Since each element would be extracted, appropriate comparator would be used based on the primitive type. In case of Int, IntegerLiteral Comparator would be used. Similarly for other types as well. Is my understanding correct? or Do we have a use case to compare two Variants as is? Please calrify. @RussellSpitzer @aihuaxu Please share your views. -- 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
