Kontinuation commented on PR #12667: URL: https://github.com/apache/iceberg/pull/12667#issuecomment-2817840347
> Left a few comments. I also wonder, we are not doing CONTAINS, CONTAINED_BY? This patch only supports INTERSECTS for simplicity, and it is the "catch-all" predicate that could support any other predicates. We can leave support for COVERS as future improvement. We are not going to support COVERED_BY, because I don't think we can do anything special for COVERD_BY given the bounding box statistics and a bbox as query window. -- 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
