Skip optimizing IN predicate when it contains too many values

[Jackie-Jiang]

MergeEqInFilterOptimizer maintains a set of thrift Expression for IN predicate values. It is very expensive to calculate hash code for Expression, thus we limit the values allowed to apply the optimizer

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 57.14286% with 3 lines in your changes missing coverage. Please review.

Project coverage is 63.99%. Comparing base (59551e4) to head (22f16b0).
Report is 1483 commits behind head on master.

Files with missing lines	Patch %	Lines
...ery/optimizer/filter/MergeEqInFilterOptimizer.java	57.14%	2 Missing and 1 partial ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##             master   #14670      +/-   ##
============================================
+ Coverage     61.75%   63.99%   +2.24%     
- Complexity      207     1604    +1397     
============================================
  Files          2436     2703     +267     
  Lines        133233   148926   +15693     
  Branches      20636    22820    +2184     
============================================
+ Hits          82274    95310   +13036     
- Misses        44911    46627    +1716     
- Partials       6048     6989     +941     

Flag	Coverage Δ
custom-integration1	100.00% <ø> (+99.99%)	⬆️
integration	100.00% <ø> (+99.99%)	⬆️
integration1	100.00% <ø> (+99.99%)	⬆️
integration2	0.00% <ø> (ø)
java-11	63.95% <57.14%> (+2.24%)	⬆️
java-21	63.89% <57.14%> (+2.26%)	⬆️
skip-bytebuffers-false	63.99% <57.14%> (+2.24%)	⬆️
skip-bytebuffers-true	63.85% <57.14%> (+36.12%)	⬆️
temurin	63.99% <57.14%> (+2.24%)	⬆️
unittests	63.99% <57.14%> (+2.24%)	⬆️
unittests1	56.31% <57.14%> (+9.42%)	⬆️
unittests2	34.42% <28.57%> (+6.69%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.