crepererum opened a new pull request #258: URL: https://github.com/apache/arrow-rs/pull/258
# Which issue does this PR close? Closes #254 . # Rationale for this change Up until now u64 values larger than `i64::MAX` were silently truncated to "invalid" when going through parquet. With this change we follow the C++ stack and # What changes are included in this PR? - a bit of prep (first two commits) - map all u64 values to i64 via overflow/reinterpret cast - account of logical type `is_signed` flag in statistics calculation (as per spec) # Are there any user-facing changes? Yes, users can now expect to have full u64 storage support. Old files should still work since we previously marked values larger than `i64::MAX` as invalid. -- 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]
