dwave commented on code in PR #8064: URL: https://github.com/apache/seatunnel/pull/8064#discussion_r1847995238
########## seatunnel-connectors-v2/connector-file/connector-file-base/src/test/java/org/apache/seatunnel/connectors/seatunnel/file/Reader/ExcelReadStrategyTest.java: ########## @@ -15,8 +15,9 @@ * limitations under the License. */ -package org.apache.seatunnel.connectors.seatunnel.file.writer; +package org.apache.seatunnel.connectors.seatunnel.file.Reader; Review Comment:  Excel stores the double type of the data of the time type at the bottom layer, so using double to convert back to the Date and DateTime types can now pass all test cases. And added the string matching recognition time formatting option of yyyy/M/d and yyyy-M-d in DateTimeUtils and DateUtils -- 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]
