Skip to content

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

Open
Caideyipi wants to merge 7 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 7 commits into
dev/1.3from
cp-feaeaa6-dev-1.3

Share WAL measurement schema serialization

2950882
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Jul 3, 2026 in 2m 22s

Quality Gate passed

Issues
44 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
1.6% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

Check failure on line 419 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=AZ65gxyjGiqAEoYQEp61&open=AZ65gxyjGiqAEoYQEp61&pullRequest=17922

Check warning on line 283 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

Rename "bitMaps" which hides the field declared at line 74. 

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

Check warning on line 449 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 150 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 'testDeserializeFromWALSkipsRetainedMeasurementWithNullSchema' must contain no more than '2' consecutive capital letters. 

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ65gxrAGiqAEoYQEp6R&open=AZ65gxrAGiqAEoYQEp6R&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 165 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=AZ65gxrAGiqAEoYQEp6S&open=AZ65gxrAGiqAEoYQEp6S&pullRequest=17922

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 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 398 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/plan/planner/plan/node/write/InsertNode.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=AZ8nkgByJRrNf0DR0JFK&open=AZ8nkgByJRrNf0DR0JFK&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 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 412 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/plan/planner/plan/node/write/InsertNode.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

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

Check warning on line 275 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=AZ65gxtDGiqAEoYQEp6h&open=AZ65gxtDGiqAEoYQEp6h&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 warning on line 360 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 420 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 173 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

Extract this nested ternary operation into an independent statement. 

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

Check failure on line 359 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

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 671 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=AZ65gxyjGiqAEoYQEp6y&open=AZ65gxyjGiqAEoYQEp6y&pullRequest=17922

Check warning on line 543 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/plan/statement/crud/InsertTabletStatement.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 84 to 64, Complexity from 28 to 14, Nesting Level from 5 to 2, Number of Variables from 26 to 6. 

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

Check failure on line 379 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 432 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/plan/statement/crud/InsertTabletStatement.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this method to simply inherit it. 

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ65gxw1GiqAEoYQEp6q&open=AZ65gxw1GiqAEoYQEp6q&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
View more details on SonarQubeCloud