manirajv06 commented on PR #13766: URL: https://github.com/apache/iceberg/pull/13766#issuecomment-3293064495
@smaheshwar-pltr Thanks for taking a look. Sorry for the delay. My understanding is, Unlike Inclusive, Strict would answer only if it is able to find the values deterministically. Thats the reason for returning `ROWS_MIGHT_NOT_MATCH` in the initial [cut](https://github.com/apache/iceberg/pull/2062#discussion_r555233694) as that would be more accurate from defensive stand point of view with the comment to catch the case that cannot match for sure later. I have made the changes to catch the cases that cannot match for sure using the prefix. > However, consider notStartsWith("b") with bound ["a", "z"] - the logic above would report that all rows don't start with "b", but "b" could very well be a row lying between "a" and "z". Yes, this should return true in Inclusive, but should not be in Strict. @rdblue Can you please check? Is my understanding correct? -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
