This is an automated email from the ASF dual-hosted git repository.
yiguolei 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 6bceb32df4b [Fix](jdbc-scanner) Fix jdbc scanner memory leak because
it didn't close `outputTable`. (#41041)
6bceb32df4b is described below
commit 6bceb32df4b83f502659552af772c4d681058114
Author: Qi Chen <[email protected]>
AuthorDate: Sat Sep 21 20:38:26 2024 +0800
[Fix](jdbc-scanner) Fix jdbc scanner memory leak because it didn't close
`outputTable`. (#41041)
## Proposed changes
[Fix] (jdbc-scanner) Fix jdbc scanner memory leak because it didn't
close `outputTable`.
---
.../src/main/java/org/apache/doris/jdbc/BaseJdbcExecutor.java | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git
a/fe/be-java-extensions/jdbc-scanner/src/main/java/org/apache/doris/jdbc/BaseJdbcExecutor.java
b/fe/be-java-extensions/jdbc-scanner/src/main/java/org/apache/doris/jdbc/BaseJdbcExecutor.java
index 3e681fa1519..6e389926571 100644
---
a/fe/be-java-extensions/jdbc-scanner/src/main/java/org/apache/doris/jdbc/BaseJdbcExecutor.java
+++
b/fe/be-java-extensions/jdbc-scanner/src/main/java/org/apache/doris/jdbc/BaseJdbcExecutor.java
@@ -100,6 +100,9 @@ public abstract class BaseJdbcExecutor implements
JdbcExecutor {
}
public void close() throws Exception {
+ if (outputTable != null) {
+ outputTable.close();
+ }
try {
if (stmt != null && !stmt.isClosed()) {
try {
@@ -112,8 +115,8 @@ public abstract class BaseJdbcExecutor implements
JdbcExecutor {
if (conn != null && resultSet != null) {
abortReadConnection(conn, resultSet);
}
- closeResources(resultSet, stmt, conn);
} finally {
+ closeResources(resultSet, stmt, conn);
if (config.getConnectionPoolMinSize() == 0 && hikariDataSource !=
null) {
hikariDataSource.close();
JdbcDataSource.getDataSource().getSourcesMap().remove(config.createCacheKey());
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
