nevi-me commented on pull request #8926: URL: https://github.com/apache/arrow/pull/8926#issuecomment-748497278
> I did not see any changes in the tests, which I would expect for a change in semantics. Aren't we not testing this yet, or how can I verify that it is now correct? There were existing tests, but they were ignored. So I removed the `#[ignore]` so they run. I was writing date64 as timestampmilli, but this wasn't per the spec, so I fixed that now. I also added an interval test to increase test coverage. ---------------------------------------------------------------- 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]
