This is an automated email from the ASF dual-hosted git repository.
gavinchou 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 441f28ae76c [fix](cloud) Fix double assignment in recycler_service.cpp
that cause… (#64168)
441f28ae76c is described below
commit 441f28ae76c7a33435b6e2437c95aae83b3d1f5c
Author: deardeng <[email protected]>
AuthorDate: Mon Jun 8 15:21:54 2026 +0800
[fix](cloud) Fix double assignment in recycler_service.cpp that cause…
(#64168)
Fix undefined behavior in RecyclerServiceImpl::statistics_recycle where
'to_recycle_bytes' was assigned twice in the same expression:
int64_t to_recycle_bytes = to_recycle_bytes = ...
This triggers -Werror=sequence-point on GCC and breaks cloud build.
Introduced by: #55455
---
cloud/src/recycler/recycler_service.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/cloud/src/recycler/recycler_service.cpp
b/cloud/src/recycler/recycler_service.cpp
index 79ee2edf907..b916d751925 100644
--- a/cloud/src/recycler/recycler_service.cpp
+++ b/cloud/src/recycler/recycler_service.cpp
@@ -239,7 +239,7 @@ void
RecyclerServiceImpl::statistics_recycle(StatisticsRecycleRequest& req, Meta
std::ranges::for_each(resource_types, [&](const auto& resource_type) {
int64_t to_recycle_num =
g_bvar_recycler_instance_last_round_to_recycle_num.get({id, resource_type});
- int64_t to_recycle_bytes = to_recycle_bytes =
+ int64_t to_recycle_bytes =
g_bvar_recycler_instance_last_round_to_recycle_bytes.get({id, resource_type});
ss << "Task Type: " << resource_type << "\n";
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
