YT: Post-commit fixes for throttling time

commit_hash:e23a92102b67714ab31f781cca1068fb9983357a

Author: hitsedesen

yt/yt/core/rpc/service.h

@@ -82,7 +82,8 @@ struct IServiceContext
     //! Returns time between request execution start and the moment of reply or cancellation (if it already happened).
     virtual std::optional<TDuration> GetExecutionDuration() const = 0;
 
-    //! Substract given throttle duration time from request execution time.
+    //! This duration will be subtracted from the request execution time.
+    //! Can be called multiple times; these durations are added up.
     virtual void RecordThrottling(TDuration throttleDuration) = 0;
 
     //! Returns trace context associated with request.

yt/yt/core/rpc/service_detail.cpp

@@ -635,7 +635,7 @@ class TServiceBase::TServiceContext
 
     void RecordThrottling(TDuration throttleDuration) override
     {
-        ThrottlingTime_ = ThrottlingTime_ ? *ThrottlingTime_ + throttleDuration : throttleDuration;
+        ThrottlingTime_ = ThrottlingTime_ + throttleDuration;
         if (ExecutionTime_) {
             *ExecutionTime_ -= throttleDuration;
         }
@@ -765,7 +765,7 @@ class TServiceBase::TServiceContext
     std::optional<TInstant> RunInstant_;
     std::optional<TInstant> ReplyInstant_;
     std::optional<TInstant> CancelInstant_;
-    std::optional<TDuration> ThrottlingTime_;
+    TDuration ThrottlingTime_;
 
     std::optional<TDuration> ExecutionTime_;
     std::optional<TDuration> TotalTime_;
@@ -1055,8 +1055,8 @@ class TServiceBase::TServiceContext
 
         ReplyInstant_ = NProfiling::GetInstant();
         ExecutionTime_ = RunInstant_ ? *ReplyInstant_ - *RunInstant_ : TDuration();
-        if (RunInstant_ && ThrottlingTime_) {
-            *ExecutionTime_ -= *ThrottlingTime_;
+        if (RunInstant_) {
+            *ExecutionTime_ -= ThrottlingTime_;
         }
         TotalTime_ = *ReplyInstant_ - ArriveInstant_;