This is an automated email from the ASF dual-hosted git repository.

lhotari pushed a commit to branch branch-3.0
in repository https://gitbox.apache.org/repos/asf/pulsar.git

commit 13f0aae8a93f91c8444d9f538f3cde4172a5788d
Author: Lari Hotari <[email protected]>
AuthorDate: Mon May 6 21:48:47 2024 +0300

    [fix][test] Clear MockedPulsarServiceBaseTest fields to prevent test 
runtime memory leak (#22659)
    
    (cherry picked from commit 025354ef4e733d62eee0d332edacb0b33b787da2)
---
 .../org/apache/pulsar/broker/auth/MockedPulsarServiceBaseTest.java | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git 
a/pulsar-broker/src/test/java/org/apache/pulsar/broker/auth/MockedPulsarServiceBaseTest.java
 
b/pulsar-broker/src/test/java/org/apache/pulsar/broker/auth/MockedPulsarServiceBaseTest.java
index 7380912e424..2cdb8e11da7 100644
--- 
a/pulsar-broker/src/test/java/org/apache/pulsar/broker/auth/MockedPulsarServiceBaseTest.java
+++ 
b/pulsar-broker/src/test/java/org/apache/pulsar/broker/auth/MockedPulsarServiceBaseTest.java
@@ -258,13 +258,20 @@ public abstract class MockedPulsarServiceBaseTest extends 
TestRetrySupport {
         }
         if (brokerGateway != null) {
             brokerGateway.close();
+            brokerGateway = null;
         }
         if (pulsarTestContext != null) {
             pulsarTestContext.close();
             pulsarTestContext = null;
         }
+
         resetConfig();
         onCleanup();
+
+        // clear fields to avoid test runtime memory leak, pulsarTestContext 
already handles closing of these instances
+        pulsar = null;
+        mockZooKeeper = null;
+        mockZooKeeperGlobal = null;
     }
 
     protected void onCleanup() {

Reply via email to