Fix memory leak introduce by change revert logic of lookup op.

2023-05-30 04:46:48 +00:00
parent 759f4bbfa2
commit 5af9d6c36d
1 changed files with 3 additions and 4 deletions
--- a/src/sql/das/ob_das_group_scan_op.cpp
+++ b/src/sql/das/ob_das_group_scan_op.cpp
@ -365,10 +365,9 @@ int ObGroupLookupOp::set_lookup_scan_group(int64_t group_id)
 int ObGroupLookupOp::revert_iter()
 {
  int ret = OB_SUCCESS;
-  /*
-   * ObLookupOp will call ObLocalIndexLookupOp::revert_iter();
-   * We do not need revert twice!
-  */
+  if (OB_FAIL(ObLocalIndexLookupOp::revert_iter())) {
+    LOG_WARN("revert local index lookup iter from group fail.", K(ret));
+  }
  group_iter_.reset();
  return ret;
 }