fixc

yiguolei · Nov 22, 2024 · 03b6ad5 · 03b6ad5
1 parent 9a632c1
commit 03b6ad5
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 25 deletions.
diff --git a/be/src/vec/runtime/vdatetime_value.cpp b/be/src/vec/runtime/vdatetime_value.cpp
@@ -1814,14 +1814,14 @@ VecDateTimeValue VecDateTimeValue::local_time() {
     return value;
 }
 
-void VecDateTimeValue::unchecked_set_time(uint16_t year, uint8_t month, uint8_t day, uint8_t hour,
-                                          uint8_t minute, uint8_t second) {
-    _year = year;
-    _month = month;
-    _day = day;
-    _hour = hour;
-    _minute = minute;
-    _second = second;
+void VecDateTimeValue::unchecked_set_time(uint32_t year, uint32_t month, uint32_t day,
+                                          uint32_t hour, uint32_t minute, uint32_t second) {
+    _year = (uint16_t)year;
+    _month = (uint8_t)month;
+    _day = (uint8_t)day;
+    _hour = (uint8_t)hour;
+    _minute = (uint8_t)minute;
+    _second = (uint8_t)second;
 }
 
 template <TimeUnit unit>
@@ -3413,26 +3413,26 @@ const char* DateV2Value<T>::day_name() const {
 }
 
 template <typename T>
-void DateV2Value<T>::unchecked_set_time(uint16_t year, uint8_t month, uint8_t day, uint8_t hour,
-                                        uint8_t minute, uint8_t second, uint32_t microsecond) {
-    date_v2_value_.year_ = year;
-    date_v2_value_.month_ = month;
-    date_v2_value_.day_ = day;
+void DateV2Value<T>::unchecked_set_time(uint32_t year, uint32_t month, uint32_t day, uint32_t hour,
+                                        uint32_t minute, uint32_t second, uint32_t microsecond) {
+    date_v2_value_.year_ = (uint16_t)year;
+    date_v2_value_.month_ = (uint8_t)month;
+    date_v2_value_.day_ = (uint8_t)day;
     if constexpr (is_datetime) {
-        date_v2_value_.hour_ = hour;
-        date_v2_value_.minute_ = minute;
-        date_v2_value_.second_ = second;
+        date_v2_value_.hour_ = (uint8_t)hour;
+        date_v2_value_.minute_ = (uint8_t)minute;
+        date_v2_value_.second_ = (uint8_t)second;
         date_v2_value_.microsecond_ = microsecond;
     }
 }
 
 template <typename T>
-void DateV2Value<T>::unchecked_set_time(uint8_t hour, uint8_t minute, uint8_t second,
+void DateV2Value<T>::unchecked_set_time(uint32_t hour, uint32_t minute, uint32_t second,
                                         uint32_t microsecond) {
     if constexpr (is_datetime) {
-        date_v2_value_.hour_ = hour;
-        date_v2_value_.minute_ = minute;
-        date_v2_value_.second_ = second;
+        date_v2_value_.hour_ = (uint8_t)hour;
+        date_v2_value_.minute_ = (uint8_t)minute;
+        date_v2_value_.second_ = (uint8_t)second;
         date_v2_value_.microsecond_ = microsecond;
     } else {
         LOG(FATAL) << "Invalid operation 'set_time' for date!";

diff --git a/be/src/vec/runtime/vdatetime_value.h b/be/src/vec/runtime/vdatetime_value.h
@@ -436,8 +436,8 @@ class VecDateTimeValue { // Now this type is a temp solution with little changes
         return true;
     }
 
-    void unchecked_set_time(uint16_t year, uint8_t month, uint8_t day, uint8_t hour, uint8_t minute,
-                            uint8_t second);
+    void unchecked_set_time(uint32_t year, uint32_t month, uint32_t day, uint32_t hour,
+                            uint32_t minute, uint32_t second);
 
     uint32_t daynr() const { return calc_daynr(_year, _month, _day); }
 
@@ -902,10 +902,10 @@ class DateV2Value {
         return true;
     }
 
-    void unchecked_set_time(uint16_t year, uint8_t month, uint8_t day, uint8_t hour, uint8_t minute,
-                            uint8_t second, uint32_t microsecond = 0);
+    void unchecked_set_time(uint32_t year, uint32_t month, uint32_t day, uint32_t hour,
+                            uint32_t minute, uint32_t second, uint32_t microsecond = 0);
 
-    void unchecked_set_time(uint8_t hour, uint8_t minute, uint8_t second, uint32_t microsecond);
+    void unchecked_set_time(uint32_t hour, uint32_t minute, uint32_t second, uint32_t microsecond);
 
     uint32_t daynr() const {
         return calc_daynr(date_v2_value_.year(), date_v2_value_.month(), date_v2_value_.day());