Skip to content

[To dev/1.3] Pipe: Fixed the bug that Disruptor may not clear the reference & will wait long time after pipe close (#17549) #17569

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
jt2594838 merged 3 commits into from
Apr 30, 2026

del

c15c096
Select commit
Loading
Failed to load commit list.
Merged

[To dev/1.3] Pipe: Fixed the bug that Disruptor may not clear the reference & will wait long time after pipe close (#17549) #17569

del
c15c096
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Apr 28, 2026 in 6m 59s

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 110 in iotdb-core/datanode/src/test/java/org/apache/iotdb/db/pipe/source/dataregion/realtime/disruptor/DisruptorShutdownTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "TimeUnit.sleep()". 

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

Check warning on line 104 in iotdb-core/datanode/src/test/java/org/apache/iotdb/db/pipe/source/dataregion/realtime/disruptor/DisruptorShutdownTest.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=AZ3TQbuyGYAMZSw7vLLD&open=AZ3TQbuyGYAMZSw7vLLD&pullRequest=17569

Check warning on line 35 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/pipe/source/dataregion/realtime/disruptor/SequenceGroups.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

First sentence of Javadoc is missing an ending period. 

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

Check warning on line 104 in iotdb-core/datanode/src/test/java/org/apache/iotdb/db/pipe/source/dataregion/realtime/disruptor/DisruptorShutdownTest.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=AZ3TQbuyGYAMZSw7vLLE&open=AZ3TQbuyGYAMZSw7vLLE&pullRequest=17569

Check warning on line 56 in iotdb-core/datanode/src/main/java/org/apache/iotdb/db/pipe/source/dataregion/realtime/disruptor/MultiProducerSequencer.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use an "AtomicReferenceArray" instead. 

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