[GitHub] [iceberg] RussellSpitzer commented on pull request #1665: Parquet: Introduce a limit to evaluate IN predicates

[GitBox <gi...@apache.org>]

RussellSpitzer commented on pull request #1665:
URL: https://github.com/apache/iceberg/pull/1665#issuecomment-717503457


   > @rdblue @RussellSpitzer What about IN predicates in places like `InclusiveMetricsEvaluator`? Can we do there any better than a hardcoded limit? The only workable option I can think if is to have a system property but I don't think it is a good idea.
   
   I'm hoping this really isn't as big an issue for manifest checks since there should be hopefully many less manifests than rowgroups :/ so I'm not as worried there. But if we are hardcoding one place for the release we may as hardcode both.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org