Update ListingTable to use StatisticsConverter

[xinlifoobar]

Which issue does this PR close?

Closes #10923
Closes #10924

Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

[xinlifoobar]

The test behaviour changes due to Utf8 is not supported in

datafusion/datafusion/core/src/datasource/file_format/parquet.rs

Lines 298 to 376 in ea21b08

	fn summarize_min_max(
	max_values: &mut [Option<MaxAccumulator>],
	min_values: &mut [Option<MinAccumulator>],
	fields: &Fields,
	i: usize,
	stat: &ParquetStatistics,
	) {
	if !stat.has_min_max_set() {
	max_values[i] = None;
	min_values[i] = None;
	return;
	}
	match stat {
	ParquetStatistics::Boolean(s) if DataType::Boolean == *fields[i].data_type() => {
	if let Some(max_value) = &mut max_values[i] {
	max_value
	.update_batch(&[Arc::new(BooleanArray::from(vec![*s.max()]))])
	.unwrap_or_else(|_| max_values[i] = None);
	}
	if let Some(min_value) = &mut min_values[i] {
	min_value
	.update_batch(&[Arc::new(BooleanArray::from(vec![*s.min()]))])
	.unwrap_or_else(|_| min_values[i] = None);
	}
	}
	ParquetStatistics::Int32(s) if DataType::Int32 == *fields[i].data_type() => {
	if let Some(max_value) = &mut max_values[i] {
	max_value
	.update_batch(&[Arc::new(Int32Array::from_value(*s.max(), 1))])
	.unwrap_or_else(|_| max_values[i] = None);
	}
	if let Some(min_value) = &mut min_values[i] {
	min_value
	.update_batch(&[Arc::new(Int32Array::from_value(*s.min(), 1))])
	.unwrap_or_else(|_| min_values[i] = None);
	}
	}
	ParquetStatistics::Int64(s) if DataType::Int64 == *fields[i].data_type() => {
	if let Some(max_value) = &mut max_values[i] {
	max_value
	.update_batch(&[Arc::new(Int64Array::from_value(*s.max(), 1))])
	.unwrap_or_else(|_| max_values[i] = None);
	}
	if let Some(min_value) = &mut min_values[i] {
	min_value
	.update_batch(&[Arc::new(Int64Array::from_value(*s.min(), 1))])
	.unwrap_or_else(|_| min_values[i] = None);
	}
	}
	ParquetStatistics::Float(s) if DataType::Float32 == *fields[i].data_type() => {
	if let Some(max_value) = &mut max_values[i] {
	max_value
	.update_batch(&[Arc::new(Float32Array::from(vec![*s.max()]))])
	.unwrap_or_else(|_| max_values[i] = None);
	}
	if let Some(min_value) = &mut min_values[i] {
	min_value
	.update_batch(&[Arc::new(Float32Array::from(vec![*s.min()]))])
	.unwrap_or_else(|_| min_values[i] = None);
	}
	}
	ParquetStatistics::Double(s) if DataType::Float64 == *fields[i].data_type() => {
	if let Some(max_value) = &mut max_values[i] {
	max_value
	.update_batch(&[Arc::new(Float64Array::from(vec![*s.max()]))])
	.unwrap_or_else(|_| max_values[i] = None);
	}
	if let Some(min_value) = &mut min_values[i] {
	min_value
	.update_batch(&[Arc::new(Float64Array::from(vec![*s.min()]))])
	.unwrap_or_else(|_| min_values[i] = None);
	}
	}
	_ => {
	max_values[i] = None;
	min_values[i] = None;
	}
	}
	}

[alamb]

It is an improvement!

[alamb]

Thank you @xinlifoobar -- this looks like a great improvement. I really like how you structured the code

[alamb]

this is a very nice formulation