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

zykkk 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 ba0eb23db01 [fix][compile] Resolve time_point type mismatch on MacOS 
and Linux (#38049)
ba0eb23db01 is described below

commit ba0eb23db012dfebf340c161db6f20b57d0d801f
Author: zy-kkk <zhongy...@gmail.com>
AuthorDate: Thu Jul 18 14:39:50 2024 +0800

    [fix][compile] Resolve time_point type mismatch on MacOS and Linux (#38049)
    
    revert #38036
    We cannot change the time_point type directly, but use macros to
    distinguish different compilation platforms, refer to #34054
---
 common/cpp/s3_rate_limiter.cpp | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/common/cpp/s3_rate_limiter.cpp b/common/cpp/s3_rate_limiter.cpp
index 108dacd106f..3bd311bc955 100644
--- a/common/cpp/s3_rate_limiter.cpp
+++ b/common/cpp/s3_rate_limiter.cpp
@@ -22,6 +22,12 @@
 #include <chrono>
 #include <mutex>
 #include <thread>
+#if defined(__APPLE__)
+#include <ctime>
+#define CURRENT_TIME std::chrono::system_clock::now()
+#else
+#define CURRENT_TIME std::chrono::high_resolution_clock::now()
+#endif
 
 namespace doris {
 // Just 10^6.
@@ -85,8 +91,7 @@ std::pair<size_t, double> S3RateLimiter::_update_remain_token(
 
 int64_t S3RateLimiter::add(size_t amount) {
     // Values obtained under lock to be checked after release
-    auto [count_value, tokens_value] =
-            _update_remain_token(std::chrono::system_clock::now(), amount);
+    auto [count_value, tokens_value] = _update_remain_token(CURRENT_TIME, 
amount);
 
     if (_limit && count_value > _limit) {
         // CK would throw exception


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to