Replace π-related bound constants with `next_up` / `next_down`

[ding-young]

Background

Rust 1.86 stabilized f64::{next_up, next_down} and f32::{next_down, next_up}, so we can now replace several hardcoded floating-point constants using these methods. Replacing manually defined constants with std library can improve clarity and correctness.

See several functions starting from

datafusion/datafusion/common/src/scalar/mod.rs

Lines 1191 to 1202 in a089eff

	/// Returns a [`ScalarValue`] representing PI's upper bound
	pub fn new_pi_upper(datatype: &DataType) -> Result<ScalarValue> {
	// TODO: replace the constants with next_up/next_down when
	// they are stabilized: https://doc.rust-lang.org/std/primitive.f64.html#method.next_up
	match datatype {
	DataType::Float32 => Ok(ScalarValue::from(consts::PI_UPPER_F32)),
	DataType::Float64 => Ok(ScalarValue::from(consts::PI_UPPER_F64)),
	_ => {
	_internal_err!("PI_UPPER is not supported for data type: {:?}", datatype)
	}
	}
	}

and https://github.com/apache/datafusion/blob/main/datafusion/common/src/scalar/consts.rs

Solution

1. Update related functions in datafusion/common/src/scalar/mod.rs
2. Verify that the updated values match the current constants, or document any small differences that arise due to precision changes.

I think this is a good first issue :)

The TODO comment is originally added in #11584

[rthummaluru]

take

[rthummaluru]

Hi everyone! 👋 This will be my first open-source contribution, and I’m super excited to get involved with DataFusion :) I’d like to take on this issue and will start setting up my local environment today. Please let me know if there’s anything specific I should keep in mind as a first-time contributor!

[rthummaluru]

Just submitted the PR, please let me know if there's anything that needs to be changed or corrected. Thank you!