Skip to content

[To dev/1.3] Fix partial insert handling for null measurements (#17879)#17922

Open
Caideyipi wants to merge 4 commits into
dev/1.3from
cp-feaeaa6-dev-1.3
Open

[To dev/1.3] Fix partial insert handling for null measurements (#17879)#17922
Caideyipi wants to merge 4 commits into
dev/1.3from
cp-feaeaa6-dev-1.3

Fix ITs

83e1ca1
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Jun 15, 2026 in 2m 11s

Quality Gate failed

Failed conditions
5.9% Duplication on New Code (required ≤ 5%)
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Annotations

Check warning on line 184 in iotdb-core/datanode/src/test/java/org/apache/iotdb/db/queryengine/plan/planner/node/write/InsertRowNodeSerdeTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Unicode escape(s) usage should be avoided.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ65gxrAGiqAEoYQEp6U&open=AZ65gxrAGiqAEoYQEp6U&pullRequest=17922

Check warning on line 203 in iotdb-core/datanode/src/test/java/org/apache/iotdb/db/queryengine/plan/planner/node/write/InsertRowNodeSerdeTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Unicode escape(s) usage should be avoided.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ65gxrAGiqAEoYQEp6X&open=AZ65gxrAGiqAEoYQEp6X&pullRequest=17922

Check warning on line 2076 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/storageengine/dataregion/DataRegion.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

A "Brain Method" was detected. Refactor it to reduce at least one of the following metrics: LOC from 89 to 64, Complexity from 21 to 14, Nesting Level from 5 to 2, Number of Variables from 17 to 6.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ7KggkFH43TdrnQYKS1&open=AZ7KggkFH43TdrnQYKS1&pullRequest=17922

Check warning on line 163 in iotdb-core/datanode/src/test/java/org/apache/iotdb/db/queryengine/plan/planner/node/write/InsertTabletNodeSerdeTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Unicode escape(s) usage should be avoided.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ65gxsyGiqAEoYQEp6d&open=AZ65gxsyGiqAEoYQEp6d&pullRequest=17922

Check warning on line 188 in iotdb-core/datanode/src/test/java/org/apache/iotdb/db/queryengine/plan/planner/node/write/InsertRowNodeSerdeTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Abbreviation in name 'testDeserializeFromWALWithMarkedFailedMeasurementOnly' must contain no more than '2' consecutive capital letters.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ65gxrAGiqAEoYQEp6W&open=AZ65gxrAGiqAEoYQEp6W&pullRequest=17922

Check warning on line 276 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/storageengine/dataregion/memtable/AbstractWritableMemChunk.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Abbreviation in name 'serializeSchemaToWALBytes' must contain no more than '2' consecutive capital letters.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ65gxu_GiqAEoYQEp6l&open=AZ65gxu_GiqAEoYQEp6l&pullRequest=17922

Check warning on line 265 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/trigger/executor/TriggerFireVisitor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Reduce the total number of break and continue statements in this loop to use at most one.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ65gx0GGiqAEoYQEp69&open=AZ65gx0GGiqAEoYQEp69&pullRequest=17922

Check warning on line 106 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/pipe/receiver/transform/statement/PipeConvertedInsertRowStatement.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Reduce the total number of break and continue statements in this loop to use at most one.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ65gxzsGiqAEoYQEp67&open=AZ65gxzsGiqAEoYQEp67&pullRequest=17922

Check warning on line 131 in iotdb-core/datanode/src/test/java/org/apache/iotdb/db/queryengine/plan/planner/node/write/InsertTabletNodeSerdeTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Abbreviation in name 'testSerializeToWALWithoutMeasurementSchemas' must contain no more than '2' consecutive capital letters.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ65gxsyGiqAEoYQEp6a&open=AZ65gxsyGiqAEoYQEp6a&pullRequest=17922

Check warning on line 441 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/pipe/event/common/tablet/TabletInsertionDataContainer.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

A "Brain Method" was detected. Refactor it to reduce at least one of the following metrics: LOC from 148 to 64, Complexity from 41 to 14, Nesting Level from 4 to 2, Number of Variables from 29 to 6.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ65gxzhGiqAEoYQEp66&open=AZ65gxzhGiqAEoYQEp66&pullRequest=17922

Check warning on line 399 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/plan/planner/plan/node/write/InsertRowNode.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Reduce the total number of break and continue statements in this loop to use at most one.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ65gxyjGiqAEoYQEp6v&open=AZ65gxyjGiqAEoYQEp6v&pullRequest=17922

Check warning on line 147 in iotdb-core/datanode/src/test/java/org/apache/iotdb/db/queryengine/plan/planner/node/write/InsertTabletNodeSerdeTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Abbreviation in name 'testSerializeToWALWithShortBitMaps' must contain no more than '2' consecutive capital letters.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ65gxsyGiqAEoYQEp6b&open=AZ65gxsyGiqAEoYQEp6b&pullRequest=17922

Check warning on line 243 in iotdb-core/datanode/src/test/java/org/apache/iotdb/db/storageengine/dataregion/memtable/AbstractMemTablePartialInsertTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Expected @throws tag for 'IllegalPathException'.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ65gxtDGiqAEoYQEp6g&open=AZ65gxtDGiqAEoYQEp6g&pullRequest=17922

Check warning on line 631 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/plan/planner/plan/node/write/InsertRowNode.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Reduce the total number of break and continue statements in this loop to use at most one.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ65gxyjGiqAEoYQEp6x&open=AZ65gxyjGiqAEoYQEp6x&pullRequest=17922

Check warning on line 86 in iotdb-core/datanode/src/test/java/org/apache/iotdb/db/queryengine/plan/planner/node/write/InsertRowsOfOneDeviceNodeSerdeTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Distance between variable 'node' declaration and its first usage is 4, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ65gxseGiqAEoYQEp6Y&open=AZ65gxseGiqAEoYQEp6Y&pullRequest=17922

Check warning on line 1075 in iotdb-core/datanode/src/test/java/org/apache/iotdb/db/storageengine/dataregion/DataRegionTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename this variable to not match a restricted identifier.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ7KgggNH43TdrnQYKSz&open=AZ7KgggNH43TdrnQYKSz&pullRequest=17922

Check failure on line 374 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/pipe/event/common/tablet/TabletInsertionDataContainer.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to reduce its Cognitive Complexity from 16 to the 15 allowed.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ65gxzhGiqAEoYQEp65&open=AZ65gxzhGiqAEoYQEp65&pullRequest=17922

Check warning on line 459 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/plan/planner/plan/node/write/InsertRowNode.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Reduce the total number of break and continue statements in this loop to use at most one.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ65gxyjGiqAEoYQEp6w&open=AZ65gxyjGiqAEoYQEp6w&pullRequest=17922

Check warning on line 184 in iotdb-core/datanode/src/test/java/org/apache/iotdb/db/queryengine/plan/planner/node/write/InsertTabletNodeSerdeTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Unicode escape(s) usage should be avoided.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ65gxsyGiqAEoYQEp6f&open=AZ65gxsyGiqAEoYQEp6f&pullRequest=17922

Check warning on line 790 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/storageengine/dataregion/memtable/AlignedWritableMemChunk.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

A "Brain Method" was detected. Refactor it to reduce at least one of the following metrics: LOC from 66 to 64, Complexity from 25 to 14, Nesting Level from 6 to 2, Number of Variables from 24 to 6.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ65gxuiGiqAEoYQEp6k&open=AZ65gxuiGiqAEoYQEp6k&pullRequest=17922

Check warning on line 169 in iotdb-core/datanode/src/test/java/org/apache/iotdb/db/queryengine/plan/planner/node/write/InsertRowNodeSerdeTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Abbreviation in name 'testDeserializeFromWALSkipsRetainedMeasurementWithMissingValue' must contain no more than '2' consecutive capital letters.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ65gxrAGiqAEoYQEp6T&open=AZ65gxrAGiqAEoYQEp6T&pullRequest=17922

Check warning on line 127 in iotdb-core/datanode/src/test/java/org/apache/iotdb/db/queryengine/plan/planner/node/write/InsertTabletNodeSerdeTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Unicode escape(s) usage should be avoided.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ65gxsyGiqAEoYQEp6Z&open=AZ65gxsyGiqAEoYQEp6Z&pullRequest=17922

Check warning on line 942 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/plan/planner/plan/node/write/InsertTabletNode.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Abbreviation in name 'getValidMeasurementNumberForWAL' must contain no more than '2' consecutive capital letters.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ65gxxtGiqAEoYQEp6t&open=AZ65gxxtGiqAEoYQEp6t&pullRequest=17922

Check warning on line 189 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/plan/planner/plan/node/write/InsertRowNode.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Fix this call that leads to a access on a value that can be null.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ65gxyjGiqAEoYQEp62&open=AZ65gxyjGiqAEoYQEp62&pullRequest=17922

Check failure on line 398 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/plan/planner/plan/node/write/InsertRowNode.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this method to reduce its Cognitive Complexity from 18 to the 15 allowed.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ65gxyjGiqAEoYQEp60&open=AZ65gxyjGiqAEoYQEp60&pullRequest=17922