You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@iceberg.apache.org by bl...@apache.org on 2022/09/20 23:57:44 UTC
[iceberg] branch master updated: API: Remove unneeded class variable (#5805)
This is an automated email from the ASF dual-hosted git repository.
blue pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/iceberg.git
The following commit(s) were added to refs/heads/master by this push:
new ee185f3f69 API: Remove unneeded class variable (#5805)
ee185f3f69 is described below
commit ee185f3f6987c092259ffd5b42b2e611e8660f47
Author: Fokko Driesprong <fo...@apache.org>
AuthorDate: Wed Sep 21 01:57:39 2022 +0200
API: Remove unneeded class variable (#5805)
---
.../main/java/org/apache/iceberg/expressions/ManifestEvaluator.java | 5 +----
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/api/src/main/java/org/apache/iceberg/expressions/ManifestEvaluator.java b/api/src/main/java/org/apache/iceberg/expressions/ManifestEvaluator.java
index c60852dacb..fc3d394203 100644
--- a/api/src/main/java/org/apache/iceberg/expressions/ManifestEvaluator.java
+++ b/api/src/main/java/org/apache/iceberg/expressions/ManifestEvaluator.java
@@ -34,7 +34,6 @@ import org.apache.iceberg.expressions.ExpressionVisitors.BoundExpressionVisitor;
import org.apache.iceberg.types.Comparators;
import org.apache.iceberg.types.Conversions;
import org.apache.iceberg.types.Type;
-import org.apache.iceberg.types.Types.StructType;
import org.apache.iceberg.util.BinaryUtil;
/**
@@ -51,7 +50,6 @@ import org.apache.iceberg.util.BinaryUtil;
public class ManifestEvaluator {
private static final int IN_PREDICATE_LIMIT = 200;
- private final StructType struct;
private final Expression expr;
public static ManifestEvaluator forRowFilter(
@@ -66,8 +64,7 @@ public class ManifestEvaluator {
}
private ManifestEvaluator(PartitionSpec spec, Expression partitionFilter, boolean caseSensitive) {
- this.struct = spec.partitionType();
- this.expr = Binder.bind(struct, rewriteNot(partitionFilter), caseSensitive);
+ this.expr = Binder.bind(spec.partitionType(), rewriteNot(partitionFilter), caseSensitive);
}
/**