[GitHub] [iceberg] rdblue commented on a change in pull request #1678: Do not create an Evaluator in ManifestGroup unless the file filter is used

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

rdblue commented on a change in pull request #1678:
URL: https://github.com/apache/iceberg/pull/1678#discussion_r514439406



##########
File path: core/src/main/java/org/apache/iceberg/ManifestGroup.java
##########
@@ -255,7 +260,7 @@ ManifestGroup planWith(ExecutorService newExecutorService) {
                 entry -> entry.status() != ManifestEntry.Status.EXISTING);
           }
 
-          if (fileFilter != null && fileFilter != Expressions.alwaysTrue()) {
+          if (evaluator != null) {

Review comment:
       We instantiate it above so that we only create one and share it for all of the evaluations. It just cuts down on Evaluator creation and garbage collection.




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