Use f64::total_cmp instead of OrderedFloat

[tustvold]

Is your feature request related to a problem or challenge? Please describe what you are trying to do.

DataFusion uses OrderedFloat in various places to order floats, this is different from arrow-rs which makes use of https://doc.rust-lang.org/std/primitive.f32.html#method.total_cmp.

Describe the solution you'd like

Moving away from OrderedFloat will:

• Ensure consistency
• Eliminate a dependency

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Additional context
Add any other context or screenshots about the feature request here.

[alamb]

Sounds like a great idea -- thanks @tustvold

I think this would likely be a good first project for someone as there is existing test coverage and should "just" be a matter of changing around some APIs

[comphead]

@tustvold please assign this to me

[comphead]

@tustvold Looks like OrderedFloat also implements Hash and Eq for floats.
Hash required for ScalarValue, Eq required for centroids. Do you want to make our own impl for Eq,Hash for f64?

[tustvold]

I added a HashValue trait in hash_utils.rs we can likely use / make public