fallintoplace opened a new pull request, #50747:
URL: https://github.com/apache/arrow/pull/50747

   ### Rationale for this change
   
   `strptime` can accept invalid calendar dates such as `1999-02-30`. 
Converting the parsed fields directly to `sys_days` normalizes those values 
into a later month instead of rejecting them.
   
   ### What changes are included in this PR?
   
   Validate the parsed `year_month_day` before converting it to `sys_days`. Add 
regression coverage for invalid month lengths and leap-year boundaries, along 
with valid boundary dates.
   
   ### Are these changes tested?
   
   Yes. The focused `TimestampParser.StrptimeParser*` tests and the complete 
`arrow-utility-test` suite pass locally.
   
   ### Are there any user-facing changes?
   
   Invalid calendar dates passed to `strptime` are now reported as parse 
failures instead of being normalized to different dates.
   
   **This PR contains a "Critical Fix".** Invalid input could previously 
produce an incorrect timestamp.
   
   Closes #31374.
   


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

Reply via email to