add ordering

[Igosuki]

Hi guys, I'm currently trying out arrow2 with the latest master from datafusion. The following updates were necessary because TypeSignature is ordered.

P.S. : The other commits are just me removing code only needed to add Decimal support.

[codecov]

Codecov Report

Merging #726 (b67e9b4) into main (b617331) will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #726      +/-   ##
==========================================
+ Coverage   70.22%   70.26%   +0.03%     
==========================================
  Files         312      312              
  Lines       16995    17016      +21     
==========================================
+ Hits        11935    11956      +21     
  Misses       5060     5060              

Impacted Files	Coverage Δ
src/datatypes/field.rs	27.45% <ø> (ø)
src/datatypes/mod.rs	81.39% <ø> (ø)
src/datatypes/physical_type.rs	100.00% <ø> (ø)
src/compute/arithmetics/time.rs	26.60% <0.00%> (+0.91%)	⬆️
src/bitmap/utils/slice_iterator.rs	92.53% <0.00%> (+1.49%)	⬆️
src/compute/boolean_kleene.rs	76.19% <0.00%> (+4.76%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b617331...b67e9b4. Read the comment docs.

[jorgecarleitao]

Thanks for this PR. Could you check #379?

[Igosuki]

Ok, so the requirement for PartialOrd is lifted so we can close this !

[jorgecarleitao]

Thanks @Igosuki , and also for all your work in the datafusion migration 🙇

[Igosuki]

👍

…

On Tue, Jan 11, 2022 at 10:00 PM Jorge Leitao ***@***.***> wrote: Thanks @Igosuki <https://github.com/Igosuki> , and also for all your work in the datafusion migration 🙇 — Reply to this email directly, view it on GitHub <#726 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AADDFBW2NL6EN2RYMTF63B3UVSK7TANCNFSM5LEJUVHQ> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>. You are receiving this because you were mentioned.Message ID: ***@***.***>