(incubator-gluten) branch main updated: [GLUTEN-9383][VL][FOLLOWUP] Improve the logic of exception handling (#9461)

[ma...@apache.org]

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

marong pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/incubator-gluten.git


The following commit(s) were added to refs/heads/main by this push:
     new bf7c712ac7 [GLUTEN-9383][VL][FOLLOWUP] Improve the logic of exception handling (#9461)
bf7c712ac7 is described below

commit bf7c712ac7addb1c76cbe65c012ad5bfefcac182
Author: Zhen Wang <64...@qq.com>
AuthorDate: Wed Apr 30 17:03:14 2025 +0800

    [GLUTEN-9383][VL][FOLLOWUP] Improve the logic of exception handling (#9461)
---
 cpp/velox/memory/VeloxMemoryManager.cc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/cpp/velox/memory/VeloxMemoryManager.cc b/cpp/velox/memory/VeloxMemoryManager.cc
index c8eec9dde8..563bb881d3 100644
--- a/cpp/velox/memory/VeloxMemoryManager.cc
+++ b/cpp/velox/memory/VeloxMemoryManager.cc
@@ -170,10 +170,10 @@ class ListenableArbitrator : public velox::memory::MemoryArbitrator {
     auto neededBytes = velox::bits::roundUp(bytes - reclaimedFreeBytes, memoryPoolTransferCapacity_);
     try {
       listener_->allocationChanged(neededBytes);
-    } catch (const std::exception& e) {
+    } catch (const std::exception&) {
       // if allocationChanged failed, we need to free the reclaimed bytes
       listener_->allocationChanged(-reclaimedFreeBytes);
-      throw e;
+      std::rethrow_exception(std::current_exception());
     }
     auto ret = growPool(pool, reclaimedFreeBytes + neededBytes, bytes);
     VELOX_CHECK(


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@gluten.apache.org
For additional commands, e-mail: commits-help@gluten.apache.org