jonathanc-n commented on code in PR #13564:
URL: https://github.com/apache/datafusion/pull/13564#discussion_r1864370397
##########
datafusion/core/tests/fuzz_cases/aggregation_fuzzer/data_generator.rs:
##########
@@ -87,7 +87,12 @@ impl DatasetGeneratorConfig {
.iter()
.filter_map(|d| {
if d.column_type.is_numeric()
- && !matches!(d.column_type, DataType::Float32 |
DataType::Float64)
+ && !matches!(
Review Comment:
> @alamb thanks for the tip, I reverted that change! For the `test_sum` fuzz
test, I removed it the same way I did for the Float types due to casting to a
DateType. This was the error I got after executing with a backtrace (many more
of this same error): `ERROR: Cast error: Failed to convert 39087111289254881.41
to datetime for Timestamp(Millisecond, None)`
I'm getting this, should I still remove it?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
