Re: [PR] [fix](arrow) Fix UT `DataTypeSerDeArrowTest` of Date type [doris]

[via GitHub]

xinyiZzz commented on code in PR #48398:
URL: https://github.com/apache/doris/pull/48398#discussion_r1976832925


##########
be/test/vec/data_types/serde/data_type_serde_arrow_test.cpp:
##########
@@ -300,19 +302,18 @@ void serialize_and_deserialize_arrow_test() {
         } break;
         case TYPE_DATETIMEV2: // uint64
         {
-            // 2022-01-01 11:11:11.111
             auto column_vector_datetimev2 = 
vectorized::ColumnVector<vectorized::UInt64>::create();
             //                auto& datetimev2_data = 
column_vector_datetimev2->get_data();
             DateV2Value<DateTimeV2ValueType> value;
-            string date_literal = "2022-01-01 11:11:11.111";
-            value.from_date_str(date_literal.c_str(), date_literal.size());
+            string date_literal = "2022-01-01 11-11-11.111";

Review Comment:
   Because the string parsing failed before, I looked at the default format in 
the code:
   
![image](https://github.com/user-attachments/assets/50e1c110-b22e-46dd-8104-d107013592b1)
   "YYYY-MM-DD HH:MM:DD.FFFFFF" is the expected datatime format?



-- 
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: commits-unsubscr...@doris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


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