Skip to content

Commit e405c8e

Browse files
committed
readyset-logictest: Don't output entire query in Antithesis details
The actual failure reason is at the beginning of the message, and usually followed by the entire failing query. Since that can get massive when fuzzing, and the query gets written out as part of the script, it's useless to include the full message in the details. Change-Id: I6a6a69646df7eb9a006a1d384c9610c6fb1e3946 Reviewed-on: https://gerrit.readyset.name/c/readyset/+/9722 Reviewed-by: Sandaruwan Gunathilake <sandaruwan@readyset.io> Tested-by: Buildkite CI
1 parent 3583792 commit e405c8e

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

readyset-logictest/src/main.rs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -643,7 +643,9 @@ impl Fuzz {
643643
json!({
644644
"failure_kind": "failing_query",
645645
"extract_file": path.to_string_lossy(),
646-
"reason": reason.message(),
646+
// Truncate reason (which includes the query) because especially large
647+
// queries are useless to log here. The query will be in the file.
648+
"reason": reason.message()[..256],
647649
}),
648650
)
649651
}

0 commit comments

Comments
 (0)