Skip to content

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

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

spotless

8d742e9
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Jun 12, 2026 in 2m 9s

Quality Gate failed

Failed conditions
6.1% 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 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 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 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 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 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 290 in iotdb-core/datanode/src/test/java/org/apache/iotdb/db/storageengine/dataregion/memtable/MemChunkDeserializeTest.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=AZ65gxtQGiqAEoYQEp6i&open=AZ65gxtQGiqAEoYQEp6i&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 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 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 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 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 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=AZ65gxrAGiqAEoYQEp6V&open=AZ65gxrAGiqAEoYQEp6V&pullRequest=17922

Check warning on line 191 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/plan/analyze/schema/NormalSchemaFetcher.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 169 to 64, Complexity from 25 to 14, Nesting Level from 3 to 2, Number of Variables from 39 to 6. 

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ65gxwVGiqAEoYQEp6n&open=AZ65gxwVGiqAEoYQEp6n&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 733 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/storageengine/dataregion/memtable/TsFileProcessor.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 79 to 64, Complexity from 20 to 14, Nesting Level from 4 to 2, Number of Variables from 22 to 6. 

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ65gxv0GiqAEoYQEp6m&open=AZ65gxv0GiqAEoYQEp6m&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 987 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 'shouldSerializeMeasurementToWAL' must contain no more than '2' consecutive capital letters. 

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