Skip to content

[Misc] Add generic extension points to reduce cross-branch conflicts - #18683

Merged
JackieTien97 merged 20 commits into
masterfrom
sync-generic-changes
Oct 8, 2026
Merged

[Misc] Add generic extension points to reduce cross-branch conflicts#18683
JackieTien97 merged 20 commits into
masterfrom
sync-generic-changes

[ConfigNode] Reject user/role profiles written by a newer version

a8c7bc9
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Oct 8, 2026 in 3m 7s

Quality Gate failed

Failed conditions
B 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 1055 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/plan/execution/config/TableConfigTaskVisitor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Line is longer than 100 characters (found 106). 

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AaEZnz-HdwEzUswl_Ly0&open=AaEZnz-HdwEzUswl_Ly0&pullRequest=18683

Check failure on line 45 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/lbac/AllowAllLBACAccessControl.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=AaEZn0SkdwEzUswl_LzG&open=AaEZn0SkdwEzUswl_LzG&pullRequest=18683

Check warning on line 338 in iotdb-core/confignode/src/main/java/org/apache/iotdb/confignode/persistence/schema/CNPhysicalPlanGenerator.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Expected @throws tag for 'IOException'. 

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

Check warning on line 171 in iotdb-core/confignode/src/main/java/org/apache/iotdb/confignode/procedure/impl/schema/table/AbstractSetPropertiesProcedure.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this "switch" statement by "if" statements to increase readability. 

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

Check warning on line 407 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/plan/relational/sql/util/DataNodeSqlFormatter.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this "switch" statement by "if" statements to increase readability. 

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

Check warning on line 50 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/plan/relational/information/AdditionalInformationSchemaProvider.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Single-line Javadoc comment should be multi-line. 

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

Check warning on line 46 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/lbac/AllowAllLBACAccessControl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

'}' at column 98 should be alone on a line. 

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

Check warning on line 36 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/lbac/ILBACAccessControl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

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

Check warning on line 26 in iotdb-core/node-commons/src/main/java/org/apache/iotdb/commons/lbac/LBACRuntimeException.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

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

Check warning on line 35 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/lbac/AllowAllLBACAccessControl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

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

Check warning on line 215 in iotdb-core/confignode/src/main/java/org/apache/iotdb/confignode/persistence/schema/CNPhysicalPlanGenerator.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Line is longer than 100 characters (found 105). 

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

Check warning on line 42 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/lbac/AllowAllLBACAccessControl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

'}' at column 40 should be alone on a line. 

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

Check warning on line 124 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/plan/analyze/schema/SchemaValidator.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use a stream instead of this loop. 

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

Check warning on line 184 in iotdb-core/node-commons/src/main/java/org/apache/iotdb/commons/auth/role/LocalFileRoleAccessor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused method parameter "role". 

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

Check warning on line 32 in iotdb-core/node-commons/src/main/java/org/apache/iotdb/commons/lbac/LBACAccessDeniedException.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

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

Check warning on line 171 in iotdb-core/confignode/src/main/java/org/apache/iotdb/confignode/procedure/impl/schema/table/AbstractSetPropertiesProcedure.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

switch without "default" clause. 

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

Check warning on line 106 in iotdb-core/node-commons/src/main/java/org/apache/iotdb/commons/auth/role/LocalFileRoleAccessor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Single-line Javadoc comment should be multi-line. 

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

Check warning on line 135 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/auth/AuthorityChecker.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

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

Check warning on line 63 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/lbac/ILBACAccessControl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

At-clause should have a non-empty description. 

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

Check warning on line 5430 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/plan/execution/config/executor/ClusterConfigTaskExecutor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Extract this nested try block into a separate method. 

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

Check warning on line 52 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/plan/relational/information/AdditionalInformationSchemaProvider.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Return an empty collection instead of null. 

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

Check warning on line 73 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/lbac/ILBACAccessControl.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

'}' at column 30 should be alone on a line. 

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

Check warning on line 337 in iotdb-core/confignode/src/main/java/org/apache/iotdb/confignode/persistence/schema/CNPhysicalPlanGenerator.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove these unused method parameters "granteeName", "isUser". 

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

Check warning on line 32 in iotdb-core/node-commons/src/main/java/org/apache/iotdb/commons/lbac/LBACAccessDeniedException.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This class has 6 parents which is greater than 5 authorized. 

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

Check failure on line 5508 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/plan/execution/config/executor/ClusterConfigTaskExecutor.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use static access with "org.apache.iotdb.commons.client.AbstractConfigNodeInfo" for "CONFIG_REGION_ID". 

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