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

rong pushed a commit to branch query-memory-leak
in repository https://gitbox.apache.org/repos/asf/iotdb.git

commit 83c34ff2ebfdd96e1a7d7b371b34ba4d1b9595fe
Author: Steve Yurong Su <[email protected]>
AuthorDate: Mon Aug 2 12:37:41 2021 +0800

    fix wrong return status
---
 server/src/main/java/org/apache/iotdb/db/service/TSServiceImpl.java | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/server/src/main/java/org/apache/iotdb/db/service/TSServiceImpl.java 
b/server/src/main/java/org/apache/iotdb/db/service/TSServiceImpl.java
index 7073d98..38aa59e 100644
--- a/server/src/main/java/org/apache/iotdb/db/service/TSServiceImpl.java
+++ b/server/src/main/java/org/apache/iotdb/db/service/TSServiceImpl.java
@@ -310,10 +310,10 @@ public class TSServiceImpl implements TSIService.Iface {
         } else {
           sessionManager.closeStatement(req.sessionId, req.statementId);
         }
+        return RpcUtils.getStatus(TSStatusCode.SUCCESS_STATUS);
+      } else {
         return RpcUtils.getStatus(
             TSStatusCode.CLOSE_OPERATION_ERROR, "statement id not set by 
client.");
-      } else {
-        return RpcUtils.getStatus(TSStatusCode.SUCCESS_STATUS);
       }
     } catch (Exception e) {
       return onNPEOrUnexpectedException(

Reply via email to