Merge pull request #11701 from Mytherin/improvestatementreduction

Minor improvements to sql_reduce script
duckdb · Apr 18, 2024 · f6e169b · f6e169b
2 parents 8c06530 + fbeeb5e
commit f6e169b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/extension/sqlsmith/statement_simplifier.cpp b/extension/sqlsmith/statement_simplifier.cpp
@@ -132,6 +132,8 @@ void StatementSimplifier::Simplify(unique_ptr<TableRef> &ref) {
 		SimplifyOptionalExpression(cp.condition);
 		SimplifyReplace(ref, cp.left);
 		SimplifyReplace(ref, cp.right);
+		SimplifyEnum(cp.type, JoinType::INNER);
+		SimplifyEnum(cp.ref_type, JoinRefType::REGULAR);
 		break;
 	}
 	case TableReferenceType::EXPRESSION_LIST: {

diff --git a/scripts/reduce_sql.py b/scripts/reduce_sql.py
@@ -258,7 +258,7 @@ def reduce_query_log(queries, shell, max_time_seconds=300):
     print(expected_error)
     print("===================================================")
 
-    final_query = reduce(sql_query, data_load, shell, expected_error, args.max_time)
+    final_query = reduce(sql_query, data_load, shell, expected_error, int(args.max_time))
     print("Found final reduced query")
     print("===================================================")
     print(final_query)