feat!: Add TinyInt / i8 type

[Ashu999]

/closes #154
/claim #154

Rationale for this change

We already have SmallInt / i16, Int / i32, and BigInt / i64 data types supported. Adding an additional TinyInt / i8 data type would be helpful to some users and would additionally have (minor) performance benefits with those columns.

What changes are included in this PR?

Added a TinyInt variant to the various Column enum types.
Added a branch for handling TinyInt everywhere that SmallInt, Int, and BigInt are handled.

Are these changes tested?

Added unit tests covering the additional code.

[Ashu999]

@Dustin-Ray, I have a question regarding tests:

There are some tests in codebase in which: either one of integer type is tested against any other non-integer type.
(eg. SmallInt, Decimal)

should i add a similar test with TinyInt there? (like TinyInt, Decimal) ?

[Ashu999]

@Dustin-Ray changes can be reviewed now

[Dustin-Ray]

should i add a similar test with TinyInt there? (like TinyInt, Decimal) ?

In general yes, we require thorough tests to cover every case possible. The tests you linked to are surrounding OwnedColumn operations and should have some coverage for Tiny int where possible.

[Dustin-Ray]

@Dustin-Ray changes can be reviewed now

Please check that the CI passes locally, check, tests, and clippy are currently failing. Ping me when these are fixed and I will run CI again

[Ashu999]

cargo check, clippy, test passing locally. Now I'm adding more tests as discussed above.

[Dustin-Ray]

cargo check, clippy, test passing locally. Now I'm adding more tests as discussed above.

we have custom ci commands, you need to run:

cargo clippy --all-targets --all-features -- -D warnings
cargo check --all-targets --all-features
cargo test --all-features

[Ashu999]

ran above mentioned commands, they are passing.

added as many tests as possible, let me know if i missed any @Dustin-Ray

[Dustin-Ray]

ran above mentioned commands, they are passing.

great thanks, will review asap

[Dustin-Ray]

LGTM - Thanks!

@iajoiner let us know what you think then we can maybe wrap this one up

[JayWhite2357]

Thanks for the PR!
You will need to resolve the branch conflicts with main. Once that is done, I'll need to reapprove.

[Ashu999]

done @JayWhite2357

[JayWhite2357]

Thanks for the PR!

[github-actions]

🎉 This PR is included in version 0.28.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

[JayWhite2357]

/approve
@Ashu999 #229 may be a similar issue that you could tackle next

[algora-pbc]

@JayWhite2357: The claim has been successfully added to reward-all. You can visit your dashboard to complete the payment.