This is an automated email from the ASF dual-hosted git repository.
jt2594838 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/iotdb.git
The following commit(s) were added to refs/heads/master by this push:
new f709ef52ff5 Avoid eager aggregation type coverage checks on first
query (#18915)
f709ef52ff5 is described below
commit f709ef52ff57f7084d988cd0091ef4f9d9e2d2aa
Author: Hongzhi Gao <[email protected]>
AuthorDate: Tue Oct 20 11:42:23 2026 +0800
Avoid eager aggregation type coverage checks on first query (#18915)
* Avoid eager aggregation type coverage checks on first query
* Remove aggregation initialization comment
---
.../src/main/java/org/apache/iotdb/db/utils/TypeServices.java | 9 ---------
.../test/java/org/apache/iotdb/db/utils/TypeServicesTest.java | 10 ++++++++++
2 files changed, 10 insertions(+), 9 deletions(-)
diff --git
a/iotdb-core/datanode/src/main/java/org/apache/iotdb/db/utils/TypeServices.java
b/iotdb-core/datanode/src/main/java/org/apache/iotdb/db/utils/TypeServices.java
index 25e75046ea6..d44ef10bf47 100644
---
a/iotdb-core/datanode/src/main/java/org/apache/iotdb/db/utils/TypeServices.java
+++
b/iotdb-core/datanode/src/main/java/org/apache/iotdb/db/utils/TypeServices.java
@@ -1458,15 +1458,6 @@ public class TypeServices {
};
};
- static {
- EXTREME_VALUE_ACCUMULATOR_STRATEGY_SERVICE.check();
- TIME_VALUE_ACCUMULATOR_STRATEGY_SERVICE.check();
- MAX_MIN_BY_ACCUMULATOR_STRATEGY_SERVICE.check();
- EVENT_WINDOW_MANAGER_PROVIDER_SERVICE.check();
- OUTPUT_COLUMN_SIZE_PER_LINE_SERVICE.check();
- MODE_ACCUMULATOR_PROVIDER_SERVICE.check();
- }
-
private Aggregation() {
// Utility class
}
diff --git
a/iotdb-core/datanode/src/test/java/org/apache/iotdb/db/utils/TypeServicesTest.java
b/iotdb-core/datanode/src/test/java/org/apache/iotdb/db/utils/TypeServicesTest.java
index 6afb127905b..708ee48d49f 100644
---
a/iotdb-core/datanode/src/test/java/org/apache/iotdb/db/utils/TypeServicesTest.java
+++
b/iotdb-core/datanode/src/test/java/org/apache/iotdb/db/utils/TypeServicesTest.java
@@ -53,6 +53,16 @@ import static org.junit.Assert.assertTrue;
public class TypeServicesTest {
+ @Test
+ public void testAggregationServicesCoverAllTypes() {
+
TypeServices.Aggregation.EXTREME_VALUE_ACCUMULATOR_STRATEGY_SERVICE.check();
+ TypeServices.Aggregation.TIME_VALUE_ACCUMULATOR_STRATEGY_SERVICE.check();
+ TypeServices.Aggregation.MAX_MIN_BY_ACCUMULATOR_STRATEGY_SERVICE.check();
+ TypeServices.Aggregation.EVENT_WINDOW_MANAGER_PROVIDER_SERVICE.check();
+ TypeServices.Aggregation.OUTPUT_COLUMN_SIZE_PER_LINE_SERVICE.check();
+ TypeServices.Aggregation.MODE_ACCUMULATOR_PROVIDER_SERVICE.check();
+ }
+
@Test
public void testTabletBinarySerializationMatchesLegacyFormat() throws
IOException {
Binary[] values = {