jorgecarleitao opened a new pull request #9298: URL: https://github.com/apache/arrow/pull/9298
The arrow spec states that `Date32` represents a date in number of days, and `Date64` represents a date in milliseconds. However, our current implementation allows `Date32(Milliseconds)` and `Date64(Days)`. This PR fixes this by removing the `DateUnit` altogether, i.e. `Date32(_) -> Date32` and `Date64(_) -> Date64`. Please note that this has a major backward incompatibility to all code that uses this datatype. Unfortunately, there is no way to solve this out of spec without an incompatible change. :( ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
