New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Implementing Unsigned Types #1325
Conversation
Very nice, out of curiosity, how much does this increases the binary size? |
14mb with unsigned types, 13mb on master |
Did you double-check the parquet statistics for the uints are correctly converted? |
Yes! The only issue I found with the parquet files was the uint32 being converted to int64. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR! Looks great, but would like to see some more tests:
@Mytherin you got me on the storage. hehe |
Thanks, looks great now. |
Task #1023
Since the unsigned types are built-in I think the most relevant changes are in the cast operators. Let me know if I forgot any interface or if I should add any other relevant tests.