This is an automated email from the ASF dual-hosted git repository.
jackie pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new 90b437f06f Small memory leak fix (#14641)
90b437f06f is described below
commit 90b437f06fadefb85ab519f0b686e14cf2c96738
Author: zeronerdzerogeekzerocool <[email protected]>
AuthorDate: Thu Dec 12 10:34:21 2024 -0800
Small memory leak fix (#14641)
---
.../apache/pinot/core/accounting/PerQueryCPUMemAccountantFactory.java | 2 +-
pinot-spi/src/main/java/org/apache/pinot/spi/trace/Tracing.java | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git
a/pinot-core/src/main/java/org/apache/pinot/core/accounting/PerQueryCPUMemAccountantFactory.java
b/pinot-core/src/main/java/org/apache/pinot/core/accounting/PerQueryCPUMemAccountantFactory.java
index f8a5a71ea7..614bb8ff85 100644
---
a/pinot-core/src/main/java/org/apache/pinot/core/accounting/PerQueryCPUMemAccountantFactory.java
+++
b/pinot-core/src/main/java/org/apache/pinot/core/accounting/PerQueryCPUMemAccountantFactory.java
@@ -339,7 +339,7 @@ public class PerQueryCPUMemAccountantFactory implements
ThreadAccountantFactory
// clear task info + stats
threadEntry.setToIdle();
// clear threadResourceUsageProvider
- _threadResourceUsageProvider.set(null);
+ _threadResourceUsageProvider.remove();
// clear _anchorThread
super.clear();
}
diff --git a/pinot-spi/src/main/java/org/apache/pinot/spi/trace/Tracing.java
b/pinot-spi/src/main/java/org/apache/pinot/spi/trace/Tracing.java
index 1f43d5e0d7..0ac5ace74d 100644
--- a/pinot-spi/src/main/java/org/apache/pinot/spi/trace/Tracing.java
+++ b/pinot-spi/src/main/java/org/apache/pinot/spi/trace/Tracing.java
@@ -186,7 +186,7 @@ public class Tracing {
@Override
public void clear() {
- _anchorThread.set(null);
+ _anchorThread.remove();
}
@Override
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
