This is an automated email from the ASF dual-hosted git repository.
morningman pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-doris.git
The following commit(s) were added to refs/heads/master by this push:
new 101c7c1 [Bug] Fix bug that double unregister the resource pool in
runtime state (#3458)
101c7c1 is described below
commit 101c7c161d7c2acaa541f89eb8b4fd12a2a3d341
Author: Mingyu Chen <morningman....@gmail.com>
AuthorDate: Mon May 4 14:48:57 2020 +0800
[Bug] Fix bug that double unregister the resource pool in runtime state
(#3458)
The resource pool in runtime state will be automatically unregistered
when deconstructing the RuntimeState. So no need to unregister it when
closing the plan fragment executor.
---
be/src/runtime/plan_fragment_executor.cpp | 14 ++++++--------
1 file changed, 6 insertions(+), 8 deletions(-)
diff --git a/be/src/runtime/plan_fragment_executor.cpp
b/be/src/runtime/plan_fragment_executor.cpp
index f050402..b5cb4c9 100644
--- a/be/src/runtime/plan_fragment_executor.cpp
+++ b/be/src/runtime/plan_fragment_executor.cpp
@@ -553,16 +553,14 @@ void PlanFragmentExecutor::close() {
}
}
-
_exec_env->thread_mgr()->unregister_pool(_runtime_state->resource_pool());
-
{
std::stringstream ss;
- // Compute the _local_time_percent before pretty_print the
runtime_profile
- // Before add this operation, the print out like that:
- // UNION_NODE (id=0):(Active: 56.720us, non-child: 00.00%)
- // After add thie operation, the print out like that:
- // UNION_NODE (id=0):(Active: 56.720us, non-child: 82.53%)
- // We can easily know the exec node excute time without child time
consumed.
+ // Compute the _local_time_percent before pretty_print the
runtime_profile
+ // Before add this operation, the print out like that:
+ // UNION_NODE (id=0):(Active: 56.720us, non-child: 00.00%)
+ // After add thie operation, the print out like that:
+ // UNION_NODE (id=0):(Active: 56.720us, non-child: 82.53%)
+ // We can easily know the exec node excute time without child time
consumed.
_runtime_state->runtime_profile()->compute_time_in_profile();
_runtime_state->runtime_profile()->pretty_print(&ss);
LOG(INFO) << ss.str();
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
