adriangb commented on code in PR #16465:
URL: https://github.com/apache/datafusion/pull/16465#discussion_r2157671900


##########
datafusion/core/tests/fuzz_cases/sort_query_fuzz.rs:
##########
@@ -579,6 +618,7 @@ impl SortFuzzerTestGenerator {
         let with_mem_limit = !query_str.contains("LIMIT") && 
self.set_memory_limit;
 
         let ctx = self.generate_random_config(config_seed, with_mem_limit)?;
+

Review Comment:
   Revert



##########
datafusion/core/tests/fuzz_cases/sort_query_fuzz.rs:
##########
@@ -72,6 +73,44 @@ async fn sort_query_fuzzer_runner() {
     fuzzer.run().await.unwrap();
 }
 
+/// Reproduce the bug with specific seeds from the [
+/// failing test case](https://github.com/apache/datafusion/issues/16452).
+#[tokio::test(flavor = "multi_thread")]
+async fn test_reproduce_sort_query_issue_16452() {
+    // Seeds from the failing test case
+    let init_seed = 10313160656544581998u64;
+    let query_seed = 15004039071976572201u64;
+    let config_seed_1 = 11807432710583113300u64;
+    let config_seed_2 = 759937414670321802u64;
+
+    let random_seed = 1u64; // Use a fixed seed to ensure consistent behavior
+
+    println!("Creating test generator with same config as original runner...");

Review Comment:
   Do you plan on merging the test? If so maybe let's clean this up?



-- 
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: github-unsubscr...@datafusion.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org

Reply via email to