MaxGekk opened a new pull request #23717: [SPARK-26805][SQL] Eliminate double checking of stringToDate and stringToTimestamp inputs URL: https://github.com/apache/spark/pull/23717 ## What changes were proposed in this pull request? In the PR, I propose to eliminate checking of parsed segments inside of the `stringToDate` and `stringToTimestamp` because such checking is already performed while constructing *java.time* classes, in particular inside of `LocalDate` and `LocalTime`. As a consequence of removing the explicit checks, the `isInvalidDate` method is not needed any more, and it was removed from `DateTimeUtils`. ## How was this patch tested? This was tested by `DateExpressionsSuite`, `DateFunctionsSuite` and `DateTimeUtilsSuite`.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
