Skip to content

Support spill of devce entry in query process - #18493

Open
Wei-hao-Li wants to merge 11 commits into
masterfrom
deviceEntrySpill-devPr
Open

Support spill of devce entry in query process#18493
Wei-hao-Li wants to merge 11 commits into
masterfrom
deviceEntrySpill-devPr

fix according review

39de0b3
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Aug 28, 2026 in 4m 42s

Quality Gate failed

Failed conditions
E 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 442 in iotdb-core/node-commons/src/main/java/org/apache/iotdb/commons/udf/builtin/relational/tvf/FFTTableFunction.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=AaBGm3bAeMy7L8180p-t&open=AaBGm3bAeMy7L8180p-t&pullRequest=18493

Check warning on line 138 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/common/MPPQueryContext.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

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

Check warning on line 25 in iotdb-core/node-commons/src/main/java/org/apache/iotdb/commons/udf/builtin/relational/tvf/fft/DoubleFFT_1D.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Rename this class name to match the regular expression '^[A-Z][a-zA-Z0-9]*$'. 

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AaBGm3aBeMy7L8180p-U&open=AaBGm3aBeMy7L8180p-U&pullRequest=18493

Check warning on line 218 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/plan/relational/metadata/spill/DeviceEntrySpillManager.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Immediately return this expression instead of assigning it to the temporary variable "queryDirectory". 

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

Check warning on line 66 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/plan/relational/metadata/spill/DeviceEntryMaterializer.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Distance between variable 'releasedRamBytes' 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=AaAXzq6yBXGTVaHSW3db&open=AaAXzq6yBXGTVaHSW3db&pullRequest=18493

Check failure on line 2680 in iotdb-core/confignode/src/main/java/org/apache/iotdb/confignode/manager/ConfigManager.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

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

Check warning on line 331 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/execution/operator/source/relational/AbstractTableScanOperator.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this expression which always evaluates to "true" 

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AaAu_-hoH7VZS0qiySfC&open=AaAu_-hoH7VZS0qiySfC&pullRequest=18493

Check failure on line 326 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/plan/relational/metadata/spill/DeviceEntrySortedMaterializer.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this throw statement from this finally block. 

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

Check warning on line 58 in iotdb-core/node-commons/src/main/java/org/apache/iotdb/commons/udf/builtin/relational/tvf/fft/FloatFFT_1D.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unnecessary cast to "double". 

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AaBGm3adeMy7L8180p-f&open=AaBGm3adeMy7L8180p-f&pullRequest=18493

Check warning on line 544 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/common/MPPQueryContext.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

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

Check warning on line 5946 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/plan/relational/analyzer/StatementAnalyzer.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Local variable name 'nArgument' must match pattern '^[a-z]([a-z0-9][a-zA-Z0-9]*)?$'. 

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

Check warning on line 412 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/common/MPPQueryContext.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

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

Check warning on line 119 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/plan/relational/metadata/spill/DeviceEntrySortedMaterializer.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Distance between variable 'releasedRamBytes' 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=AaAXzq7jBXGTVaHSW3do&open=AaAXzq7jBXGTVaHSW3do&pullRequest=18493

Check warning on line 398 in iotdb-core/node-commons/src/main/java/org/apache/iotdb/commons/udf/builtin/relational/tvf/FFTTableFunction.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=AaBGm3bAeMy7L8180p-p&open=AaBGm3bAeMy7L8180p-p&pullRequest=18493

Check warning on line 72 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/metric/SeriesScanCostMetricSet.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

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

Check warning on line 576 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/common/MPPQueryContext.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

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

Check warning on line 498 in iotdb-core/node-commons/src/main/java/org/apache/iotdb/commons/udf/builtin/relational/tvf/FFTTableFunction.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=AaBGm3bAeMy7L8180p-x&open=AaBGm3bAeMy7L8180p-x&pullRequest=18493

Check failure on line 398 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/execution/operator/source/relational/AbstractAggTableScanOperator.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=AaAXzrElBXGTVaHSW3d6&open=AaAXzrElBXGTVaHSW3d6&pullRequest=18493

Check warning on line 97 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/statistics/QueryPlanStatistics.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

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

Check warning on line 118 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/statistics/QueryPlanStatistics.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

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

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

Check warning on line 113 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/plan/execution/config/sys/subscription/ShowSubscriptionsTask.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this usage of 'Stream.collect(Collectors.toList())' with 'Stream.toList()' and ensure that the list is unmodified. 

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

Check warning on line 82 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/queryengine/plan/relational/metadata/spill/DeviceEntryFileSpillerReader.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Cast one of the operands of this addition operation to a "long". 

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

Check warning on line 86 in iotdb-core/node-commons/src/main/java/org/apache/iotdb/commons/udf/builtin/relational/tvf/fft/FloatFFT_1D.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unnecessary cast to "double". 

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AaBGm3adeMy7L8180p-g&open=AaBGm3adeMy7L8180p-g&pullRequest=18493

Check warning on line 154 in iotdb-core/node-commons/src/main/java/org/apache/iotdb/commons/udf/builtin/relational/tvf/FFTTableFunction.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Call "Optional#isPresent()" or "!Optional#isEmpty()" before accessing the value. 

See more on https://sonarcloud.io/project/issues?id=apache_iotdb&issues=AaBGm3bAeMy7L8180p-k&open=AaBGm3bAeMy7L8180p-k&pullRequest=18493

Check warning on line 387 in iotdb-core/node-commons/src/main/java/org/apache/iotdb/commons/udf/builtin/relational/tvf/FFTTableFunction.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=AaBGm3bAeMy7L8180p-o&open=AaBGm3bAeMy7L8180p-o&pullRequest=18493
View more details on SonarQubeCloud