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

dataroaring pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/master by this push:
     new 4b093d1ef6 [Bug](point query) when prepared statement used 
lazyEvaluateRangeLocations should clear bucketSeq2locations to avoid memleak 
(#16531)
4b093d1ef6 is described below

commit 4b093d1ef6bf51d0101c0a734e4976b73069320f
Author: lihangyu <[email protected]>
AuthorDate: Thu Feb 9 14:41:07 2023 +0800

    [Bug](point query) when prepared statement used lazyEvaluateRangeLocations 
should clear bucketSeq2locations to avoid memleak (#16531)
    
    When JDBC client enable server side prepared statement, it will cache 
OlapScanNode and reuse it for performance, but each
    time call `addScanRangeLocations` will add new item to 
`bucketSeq2locations`, so the `bucketSeq2locations` lead to a memleak if 
OlapScanNode
    cached in memory
---
 fe/fe-core/src/main/java/org/apache/doris/planner/OlapScanNode.java | 1 +
 1 file changed, 1 insertion(+)

diff --git 
a/fe/fe-core/src/main/java/org/apache/doris/planner/OlapScanNode.java 
b/fe/fe-core/src/main/java/org/apache/doris/planner/OlapScanNode.java
index 75c69af4f4..ccc1d14b0e 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/planner/OlapScanNode.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/planner/OlapScanNode.java
@@ -989,6 +989,7 @@ public class OlapScanNode extends ScanNode {
         scanBackendIds.clear();
         scanTabletIds.clear();
         result.clear();
+        bucketSeq2locations.clear();
         try {
             getScanRangeLocations();
         } catch (AnalysisException e) {


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to