gavinchou commented on code in PR #56061:
URL: https://github.com/apache/doris/pull/56061#discussion_r2349533104
##########
cloud/src/meta-service/meta_service.h:
##########
@@ -977,14 +977,23 @@ class MetaServiceProxy final : public MetaService {
auto dist =
std::uniform_int_distribution(-config::idempotent_request_replay_delay_range_ms,
config::idempotent_request_replay_delay_range_ms);
int64_t sleep_ms = config::idempotent_request_replay_delay_base_ms
+ dist(rng);
+ std::string debug_string = req.ShortDebugString();
+ if constexpr (std::is_same_v<Request, GetTabletStatsRequest>) {
+ auto short_req = req;
+ if (short_req.tablet_idx_size() > 10) {
+ short_req.mutable_tablet_idx()->DeleteSubrange(10,
req.tablet_idx_size() - 10);
Review Comment:
make a copy of subrange instead of changing in place, in case of we use the
request for next retry.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
