Skip to content

[to dev/1.3] Optimize InsertTabletStatement tablet conversion (#17805)#17842

Merged
jt2594838 merged 1 commit into
dev/1.3from
backport-17805-dev-1.3
Jun 4, 2026
Merged

[to dev/1.3] Optimize InsertTabletStatement tablet conversion (#17805)#17842
jt2594838 merged 1 commit into
dev/1.3from
backport-17805-dev-1.3

Optimize InsertTabletStatement tablet conversion (#17805)

4ee196f
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Jun 4, 2026 in 1m 44s

Quality Gate passed

Issues
14 New issues
0 Accepted issues

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

See analysis details on SonarQube Cloud

Annotations

Check warning on line 758 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/pipe/receiver/protocol/thrift/IoTDBDataNodeReceiver.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

WhitespaceAround: '{' is not followed by whitespace. Empty blocks may only be represented as {} when not part of a multi-block statement (4.1.3) 

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

Check warning on line 34 in iotdb-core/datanode/src/test/java/org/apache/iotdb/db/pipe/receiver/protocol/thrift/IoTDBDataNodeReceiverTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ6Qgmy-JDVwm76JMBD0&open=AZ6Qgmy-JDVwm76JMBD0&pullRequest=17842

Check warning on line 758 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/pipe/receiver/protocol/thrift/IoTDBDataNodeReceiver.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

WhitespaceAround: '}' is not preceded with whitespace. 

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ6Qgm3uJDVwm76JMBD-&open=AZ6Qgm3uJDVwm76JMBD-&pullRequest=17842

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

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

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

Check warning on line 140 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/pipe/sink/protocol/thrift/async/handler/PipeTransferTsFileHandler.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 93 to 64, Complexity from 17 to 14, Nesting Level from 3 to 2, Number of Variables from 13 to 6. 

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

Check warning on line 258 in iotdb-core/confignode/src/main/java/org/apache/iotdb/confignode/persistence/UDFInfo.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

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

Check warning on line 206 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/pipe/sink/protocol/thrift/async/handler/PipeTransferTsFileHandler.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=AZ6Qgm12JDVwm76JMBD7&open=AZ6Qgm12JDVwm76JMBD7&pullRequest=17842

Check warning on line 67 in iotdb-core/confignode/src/test/java/org/apache/iotdb/confignode/persistence/UDFInfoTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove the declaration of thrown exception 'java.io.IOException', as it cannot be thrown from method's body. 

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

Check warning on line 503 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=AZ6Qgm1HJDVwm76JMBD5&open=AZ6Qgm1HJDVwm76JMBD5&pullRequest=17842

Check warning on line 49 in iotdb-core/node-commons/src/main/java/org/apache/iotdb/commons/executable/ReferenceCountedJarMetaKeeper.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this "Map.containsKey()" with a call to "Map.computeIfPresent()". 

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

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

Rename "times" which hides the field declared at line 70. 

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

Check warning on line 75 in iotdb-core/datanode/src/test/java/org/apache/iotdb/db/pipe/receiver/PipeStatementTsStatusVisitorTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Line is longer than 100 characters (found 101). 

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

Check warning on line 311 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/plan/statement/crud/LoadTsFileStatement.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename "databaseLevel" which hides the field declared at line 52. 

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

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

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

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ6Qgm1HJDVwm76JMBD2&open=AZ6Qgm1HJDVwm76JMBD2&pullRequest=17842
View more details on SonarQubeCloud