amitagrawalcs opened a new pull request #1401:
URL: https://github.com/apache/arrow-rs/pull/1401


   # Rationale for this change
    
   When we read date prior to 1st-Jan-1970 using parquet module, code panic by 
saying 'No such local time'. This bug exists because datatype date and 
timestamp in enum 'Field' (parquet/src/record/api.rs) is u32 and u64 which 
indicates the number of days from EPOCH DATE. In case of date being prior to 
1/1/1970, number of days will be a negative integer that can't be hold by u32 
and u64.
   
   # What changes are included in this PR?
   
   Datatype of date and timestamp in Field enum is changed to i32 and i64 
respectively.
   
   <!---
   There is no need to duplicate the description in the issue here but it is 
sometimes worth providing a summary of the individual changes in this PR.
   -->
   
   # Are there any user-facing changes?
   No
   
   <!---
   If there are user-facing changes then we may require documentation to be 
updated before approving the PR.
   -->
   


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