github-actions[bot] commented on code in PR #14390:
URL: https://github.com/apache/doris/pull/14390#discussion_r1064422343


##########
be/src/runtime/exec_env.cpp:
##########
@@ -18,10 +18,13 @@
 #include "runtime/exec_env.h"
 
 #include "gen_cpp/HeartbeatService_types.h"
+#include "util/time.h"
 
 namespace doris {
 
-ExecEnv::ExecEnv() : _is_init(false) {}
+ExecEnv::ExecEnv() : _is_init(false) {
+    _be_start_time_sec = GetCurrentTimeMicros() / 1000000;

Review Comment:
   warning: use of undeclared identifier '_be_start_time_sec' 
[clang-diagnostic-error]
   ```cpp
       _be_start_time_sec = GetCurrentTimeMicros() / 1000000;
       ^
   ```
   



##########
be/src/runtime/exec_env.h:
##########
@@ -184,6 +184,8 @@ class ExecEnv {
     HeartbeatFlags* heartbeat_flags() { return _heartbeat_flags; }
     doris::vectorized::ScannerScheduler* scanner_scheduler() { return 
_scanner_scheduler; }
 
+    int64_t be_start_time_sec() { return _be_start_time_sec; }

Review Comment:
   warning: reference to non-static member function must be called; did you 
mean to call it with no arguments? [clang-diagnostic-error]
   
   ```suggestion
       int64_t be_start_time_sec() { return _be_start_time_sec(); }
   ```
   



##########
be/src/runtime/exec_env.h:
##########
@@ -184,6 +184,8 @@
     HeartbeatFlags* heartbeat_flags() { return _heartbeat_flags; }
     doris::vectorized::ScannerScheduler* scanner_scheduler() { return 
_scanner_scheduler; }
 
+    int64_t be_start_time_sec() { return _be_start_time_sec; }

Review Comment:
   warning: use of undeclared identifier '_be_start_time_sec'; did you mean 
'be_start_time_sec'? [clang-diagnostic-error]
   
   ```suggestion
       int64_t be_start_time_sec() { return be_start_time_sec; }
   ```
   **be/src/runtime/exec_env.h:186:** 'be_start_time_sec' declared here
   ```cpp
       int64_t be_start_time_sec() { return _be_start_time_sec; }
               ^
   ```
   



-- 
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]

Reply via email to