Skip to content
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.

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

[improve][ci] Detect test thread leaks in CI builds and add tooling for resource leak investigation #153

Closed
wants to merge 6 commits into from

[improve][ci] Detect test thread leaks in CI builds and add tooling f…

9b371f4
Select commit
Loading
Failed to load commit list.
Closed

[improve][ci] Detect test thread leaks in CI builds and add tooling for resource leak investigation #153

[improve][ci] Detect test thread leaks in CI builds and add tooling f…
9b371f4
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Oct 26, 2023 in 1s

76.26% of diff hit (target 72.80%)

View this Pull Request on Codecov

76.26% of diff hit (target 72.80%)

Annotations

Check warning on line 621 in managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedLedgerFactoryImpl.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

managed-ledger/src/main/java/org/apache/bookkeeper/mledger/impl/ManagedLedgerFactoryImpl.java#L620-L621

Added lines #L620 - L621 were not covered by tests

Check warning on line 278 in pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/ClustersBase.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/ClustersBase.java#L277-L278

Added lines #L277 - L278 were not covered by tests

Check warning on line 280 in pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/ClustersBase.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/ClustersBase.java#L280

Added line #L280 was not covered by tests

Check warning on line 282 in pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/ClustersBase.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/ClustersBase.java#L282

Added line #L282 was not covered by tests

Check warning on line 364 in pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/PersistentTopicsBase.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/impl/PersistentTopicsBase.java#L364

Added line #L364 was not covered by tests

Check warning on line 1744 in pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/v1/Namespaces.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

pulsar-broker/src/main/java/org/apache/pulsar/broker/admin/v1/Namespaces.java#L1744

Added line #L1744 was not covered by tests

Check warning on line 1177 in pulsar-broker/src/main/java/org/apache/pulsar/broker/loadbalance/impl/ModularLoadManagerImpl.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

pulsar-broker/src/main/java/org/apache/pulsar/broker/loadbalance/impl/ModularLoadManagerImpl.java#L1177

Added line #L1177 was not covered by tests

Check warning on line 1381 in pulsar-broker/src/main/java/org/apache/pulsar/broker/service/AbstractTopic.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

pulsar-broker/src/main/java/org/apache/pulsar/broker/service/AbstractTopic.java#L1380-L1381

Added lines #L1380 - L1381 were not covered by tests

Check warning on line 460 in pulsar-broker/src/main/java/org/apache/pulsar/zookeeper/LocalBookkeeperEnsemble.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

pulsar-broker/src/main/java/org/apache/pulsar/zookeeper/LocalBookkeeperEnsemble.java#L460

Added line #L460 was not covered by tests

Check warning on line 478 in pulsar-broker/src/main/java/org/apache/pulsar/zookeeper/LocalBookkeeperEnsemble.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

pulsar-broker/src/main/java/org/apache/pulsar/zookeeper/LocalBookkeeperEnsemble.java#L478

Added line #L478 was not covered by tests

Check warning on line 43 in pulsar-client-admin-api/src/main/java/org/apache/pulsar/common/policies/data/ClusterPolicies.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

pulsar-client-admin-api/src/main/java/org/apache/pulsar/common/policies/data/ClusterPolicies.java#L43

Added line #L43 was not covered by tests

Check warning on line 161 in pulsar-client-admin-api/src/main/java/org/apache/pulsar/common/policies/data/Policies.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

pulsar-client-admin-api/src/main/java/org/apache/pulsar/common/policies/data/Policies.java#L161

Added line #L161 was not covered by tests

Check warning on line 158 in pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdClusters.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdClusters.java#L158

Added line #L158 was not covered by tests

Check warning on line 165 in pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdClusters.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

pulsar-client-tools/src/main/java/org/apache/pulsar/admin/cli/CmdClusters.java#L163-L165

Added lines #L163 - L165 were not covered by tests

Check warning on line 35 in pulsar-common/src/main/java/org/apache/pulsar/common/policies/data/ClusterPoliciesImpl.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

pulsar-common/src/main/java/org/apache/pulsar/common/policies/data/ClusterPoliciesImpl.java#L35

Added line #L35 was not covered by tests

Check warning on line 51 in pulsar-common/src/main/java/org/apache/pulsar/common/policies/data/ClusterPoliciesImpl.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

pulsar-common/src/main/java/org/apache/pulsar/common/policies/data/ClusterPoliciesImpl.java#L51

Added line #L51 was not covered by tests

Check warning on line 58 in pulsar-common/src/main/java/org/apache/pulsar/common/policies/data/ClusterPoliciesImpl.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

pulsar-common/src/main/java/org/apache/pulsar/common/policies/data/ClusterPoliciesImpl.java#L56-L58

Added lines #L56 - L58 were not covered by tests

Check warning on line 64 in pulsar-common/src/main/java/org/apache/pulsar/common/policies/data/ClusterPoliciesImpl.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

pulsar-common/src/main/java/org/apache/pulsar/common/policies/data/ClusterPoliciesImpl.java#L63-L64

Added lines #L63 - L64 were not covered by tests

Check warning on line 67 in pulsar-common/src/main/java/org/apache/pulsar/common/policies/data/ClusterPoliciesImpl.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

pulsar-common/src/main/java/org/apache/pulsar/common/policies/data/ClusterPoliciesImpl.java#L66-L67

Added lines #L66 - L67 were not covered by tests

Check warning on line 71 in pulsar-common/src/main/java/org/apache/pulsar/common/policies/data/ClusterPoliciesImpl.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

pulsar-common/src/main/java/org/apache/pulsar/common/policies/data/ClusterPoliciesImpl.java#L70-L71

Added lines #L70 - L71 were not covered by tests

Check warning on line 76 in pulsar-common/src/main/java/org/apache/pulsar/common/policies/data/ClusterPoliciesImpl.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

pulsar-common/src/main/java/org/apache/pulsar/common/policies/data/ClusterPoliciesImpl.java#L75-L76

Added lines #L75 - L76 were not covered by tests

Check warning on line 80 in pulsar-common/src/main/java/org/apache/pulsar/common/policies/data/ClusterPoliciesImpl.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

pulsar-common/src/main/java/org/apache/pulsar/common/policies/data/ClusterPoliciesImpl.java#L80

Added line #L80 was not covered by tests

Check warning on line 68 in pulsar-common/src/main/java/org/apache/pulsar/common/util/ThreadDumpUtil.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

pulsar-common/src/main/java/org/apache/pulsar/common/util/ThreadDumpUtil.java#L68

Added line #L68 was not covered by tests