Skip to content

Fixed multiple configNode bugs - #17609

Open
Caideyipi wants to merge 13 commits into
masterfrom
config-bug
Open

Fixed multiple configNode bugs#17609
Caideyipi wants to merge 13 commits into
masterfrom
config-bug

Fix NodeManager Sonar issues

a12b657
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Jul 31, 2026 in 3m 33s

Quality Gate failed

Failed conditions
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 205 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/pipe/processor/twostage/plugin/TwoStageCountProcessor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Line is longer than 100 characters (found 112).

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ-2Z_RL049OftIvJ8Q8&open=AZ-2Z_RL049OftIvJ8Q8&pullRequest=17609

Check warning on line 508 in iotdb-core/confignode/src/main/java/org/apache/iotdb/confignode/procedure/impl/pipe/AbstractOperatePipeProcedureV2.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this instanceof check and cast with 'instanceof AbstractOperatePipeProcedureV2 abstractoperatepipeprocedurev2'

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ-xIGrwojDc0-73zfPx&open=AZ-xIGrwojDc0-73zfPx&pullRequest=17609

Check warning on line 566 in iotdb-core/confignode/src/main/java/org/apache/iotdb/confignode/procedure/impl/pipe/AbstractOperatePipeProcedureV2.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

'switch rcurly' has incorrect indentation level 8, expected level should be 4.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ-xIGrwojDc0-73zfQE&open=AZ-xIGrwojDc0-73zfQE&pullRequest=17609

Check warning on line 513 in iotdb-core/confignode/src/main/java/org/apache/iotdb/confignode/procedure/impl/pipe/AbstractOperatePipeProcedureV2.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Line is longer than 100 characters (found 102).

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ-xIGrwojDc0-73zfP8&open=AZ-xIGrwojDc0-73zfP8&pullRequest=17609

Check failure on line 111 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/plan/relational/security/AllowAllAccessControl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ-xIGnkojDc0-73zfPY&open=AZ-xIGnkojDc0-73zfPY&pullRequest=17609

Check warning on line 61 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/audit/DNAuditLogger.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Simplify this regular expression to reduce its runtime, as it has super-linear performance due to backtracking.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ-xIGqXojDc0-73zfPp&open=AZ-xIGqXojDc0-73zfPp&pullRequest=17609

Check warning on line 114 in iotdb-core/confignode/src/main/java/org/apache/iotdb/confignode/procedure/impl/pipe/AbstractOperatePipeProcedureV2.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a thread-safe type; adding "volatile" is not enough to make this field thread-safe.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ-xIGrwojDc0-73zfPv&open=AZ-xIGrwojDc0-73zfPv&pullRequest=17609

Check warning on line 480 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/pipe/event/common/tsfile/parser/scan/TsFileInsertionEventScanParser.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 134 to 64, Complexity from 34 to 14, Nesting Level from 4 to 2, Number of Variables from 8 to 6.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ-xIGpDojDc0-73zfPZ&open=AZ-xIGpDojDc0-73zfPZ&pullRequest=17609

Check warning on line 72 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/audit/PasswordChangeAuditTask.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Catch Exception instead of Error.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ-xIGqIojDc0-73zfPn&open=AZ-xIGqIojDc0-73zfPn&pullRequest=17609

Check warning on line 115 in iotdb-core/confignode/src/main/java/org/apache/iotdb/confignode/procedure/impl/pipe/AbstractOperatePipeProcedureV2.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a thread-safe type; adding "volatile" is not enough to make this field thread-safe.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ-xIGrwojDc0-73zfPw&open=AZ-xIGrwojDc0-73zfPw&pullRequest=17609

Check warning on line 3799 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/protocol/thrift/impl/ClientRPCServiceImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Expected @throws tag for 'IllegalStateException'.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ-2Z_J7049OftIvJ8Q6&open=AZ-2Z_J7049OftIvJ8Q6&pullRequest=17609

Check warning on line 283 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/pipe/agent/task/builder/PipeDataNodeTaskBuilder.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ-2Z_Nr049OftIvJ8Q7&open=AZ-2Z_Nr049OftIvJ8Q7&pullRequest=17609

Check warning on line 116 in iotdb-core/confignode/src/main/java/org/apache/iotdb/confignode/procedure/env/ConfigNodeProcedureEnv.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=AZ-xIGssojDc0-73zfQI&open=AZ-xIGssojDc0-73zfQI&pullRequest=17609

Check warning on line 505 in iotdb-core/confignode/src/main/java/org/apache/iotdb/confignode/procedure/impl/pipe/AbstractOperatePipeProcedureV2.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Line is longer than 100 characters (found 117).

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ-xIGrwojDc0-73zfP7&open=AZ-xIGrwojDc0-73zfP7&pullRequest=17609

Check warning on line 471 in iotdb-core/confignode/src/main/java/org/apache/iotdb/confignode/procedure/impl/pipe/AbstractOperatePipeProcedureV2.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Line is longer than 100 characters (found 126).

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ-xIGrwojDc0-73zfP2&open=AZ-xIGrwojDc0-73zfP2&pullRequest=17609

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

Method has 10 parameters, which is greater than 7 authorized.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ-xIGklojDc0-73zfPP&open=AZ-xIGklojDc0-73zfPP&pullRequest=17609

Check warning on line 189 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/audit/DNAuditLogger.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Merge the previous cases into this one using comma-separated label.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ-xIGqXojDc0-73zfPo&open=AZ-xIGqXojDc0-73zfPo&pullRequest=17609

Check warning on line 478 in iotdb-core/confignode/src/main/java/org/apache/iotdb/confignode/procedure/impl/pipe/AbstractOperatePipeProcedureV2.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Line is longer than 100 characters (found 130).

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ-xIGrwojDc0-73zfP4&open=AZ-xIGrwojDc0-73zfP4&pullRequest=17609

Check failure on line 267 in iotdb-core/confignode/src/main/java/org/apache/iotdb/confignode/procedure/impl/pipe/AbstractOperatePipeProcedureV2.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Fix this access that will throw a NullPointerException when executed.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ-xIGrwojDc0-73zfPy&open=AZ-xIGrwojDc0-73zfPy&pullRequest=17609

Check warning on line 551 in iotdb-core/confignode/src/main/java/org/apache/iotdb/confignode/procedure/store/ProcedureFactory.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this instanceof check and cast with 'instanceof AlterTopicProcedure altertopicprocedure'

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ-2Z_eT049OftIvJ8Q9&open=AZ-2Z_eT049OftIvJ8Q9&pullRequest=17609

Check warning on line 1271 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/utils/datastructure/AlignedTVList.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

All overloaded methods should be placed next to each other. Placing non-overloaded methods in between overloaded methods with the same type is a violation. Previous overloaded method located at line '1,253'.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ-xIGpdojDc0-73zfPi&open=AZ-xIGpdojDc0-73zfPi&pullRequest=17609

Check warning on line 325 in iotdb-core/node-commons/src/main/java/org/apache/iotdb/commons/pipe/source/IoTDBNonDataRegionSource.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Line is longer than 100 characters (found 113).

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ-2Z-Zh049OftIvJ8Q1&open=AZ-2Z-Zh049OftIvJ8Q1&pullRequest=17609

Check warning on line 3714 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/protocol/thrift/impl/ClientRPCServiceImpl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Expected @throws tag for 'IllegalStateException'.

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AZ-2Z_J7049OftIvJ8Q5&open=AZ-2Z_J7049OftIvJ8Q5&pullRequest=17609

Check warning on line 94 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/audit/DNAuditLogger.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Distance between variable 'type' declaration and its first usage is 6, 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=AZ-xIGqXojDc0-73zfPt&open=AZ-xIGqXojDc0-73zfPt&pullRequest=17609

Check warning on line 93 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/audit/DNAuditLogger.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Distance between variable 'address' declaration and its first usage is 6, 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=AZ-xIGqXojDc0-73zfPs&open=AZ-xIGqXojDc0-73zfPs&pullRequest=17609