This is an automated email from the ASF dual-hosted git repository.
fokko 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 2f15f73cff Build: Fix minor errorprone warnings during compile (#6983)
2f15f73cff is described below
commit 2f15f73cff085f00f6a597f3fd44e39c1ddbd796
Author: Ajantha Bhat <[email protected]>
AuthorDate: Thu Mar 2 14:53:17 2023 +0530
Build: Fix minor errorprone warnings during compile (#6983)
---
.../main/java/org/apache/iceberg/expressions/AggregateEvaluator.java | 1 +
api/src/main/java/org/apache/iceberg/expressions/ValueAggregate.java | 1 +
baseline.gradle | 2 ++
3 files changed, 4 insertions(+)
diff --git
a/api/src/main/java/org/apache/iceberg/expressions/AggregateEvaluator.java
b/api/src/main/java/org/apache/iceberg/expressions/AggregateEvaluator.java
index 7c03796c24..204a4e7096 100644
--- a/api/src/main/java/org/apache/iceberg/expressions/AggregateEvaluator.java
+++ b/api/src/main/java/org/apache/iceberg/expressions/AggregateEvaluator.java
@@ -108,6 +108,7 @@ public class AggregateEvaluator {
this.values = values;
}
+ @Override
public int size() {
return values.length;
}
diff --git
a/api/src/main/java/org/apache/iceberg/expressions/ValueAggregate.java
b/api/src/main/java/org/apache/iceberg/expressions/ValueAggregate.java
index b2e68136dd..5db1323bec 100644
--- a/api/src/main/java/org/apache/iceberg/expressions/ValueAggregate.java
+++ b/api/src/main/java/org/apache/iceberg/expressions/ValueAggregate.java
@@ -33,6 +33,7 @@ class ValueAggregate<T> extends BoundAggregate<T, T> {
return term().eval(struct);
}
+ @Override
public T eval(DataFile file) {
valueStruct.setValue(evaluateRef(file));
return term().eval(valueStruct);
diff --git a/baseline.gradle b/baseline.gradle
index d8e0b58134..ee2d72dee3 100644
--- a/baseline.gradle
+++ b/baseline.gradle
@@ -86,6 +86,8 @@ subprojects {
'-Xep:PreferSafeLogger:OFF',
// Enforce safe string splitting
'-Xep:StringSplitter:ERROR',
+ // Enforce missing override
+ '-Xep:MissingOverride:ERROR',
)
}
}