hutiefang76 opened a new pull request, #8338: URL: https://github.com/apache/paimon/pull/8338
### Purpose Fixes #7230. `FileIndexPredicate#getRequiredNames` visited each child predicate twice while collecting required field names. This could make nested compound predicates traverse much more work than needed. This patch reuses the existing `PredicateVisitor.collectFieldNames` helper instead of keeping a duplicate visitor implementation in `FileIndexPredicate`. ### Tests - `JAVA_HOME=$(/usr/libexec/java_home -v 17) mvn -pl paimon-common -Pfast-build -Dtest=FileIndexPredicateTest#testGetRequiredNamesVisitsEachChildOnce test` - `JAVA_HOME=$(/usr/libexec/java_home -v 17) mvn -pl paimon-common -Pfast-build -Dtest=FileIndexFormatFormatTest test` - `JAVA_HOME=$(/usr/libexec/java_home -v 17) mvn -pl paimon-common -Pfast-build -Dtest=FileIndexPredicateTest,FileIndexFormatFormatTest test` - `git diff --check` -- 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]
