This is an automated email from the ASF dual-hosted git repository.

github-bot pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/datafusion.git


The following commit(s) were added to refs/heads/main by this push:
     new a45192084e chore: remove dialect settings in SLT tests that are 
outdated (#17807)
a45192084e is described below

commit a45192084e95a2d629b7cbf88b43133d8d6fb814
Author: Jeffrey Vo <[email protected]>
AuthorDate: Mon Sep 29 13:43:14 2025 +1000

    chore: remove dialect settings in SLT tests that are outdated (#17807)
---
 datafusion/sqllogictest/test_files/aggregate.slt         | 16 ----------------
 .../sqllogictest/test_files/aggregate_skip_partial.slt   |  7 -------
 datafusion/sqllogictest/test_files/create_function.slt   |  5 -----
 datafusion/sqllogictest/test_files/expr.slt              |  6 ------
 datafusion/sqllogictest/test_files/group_by.slt          |  8 --------
 5 files changed, 42 deletions(-)

diff --git a/datafusion/sqllogictest/test_files/aggregate.slt 
b/datafusion/sqllogictest/test_files/aggregate.slt
index 2b4daff9e6..9d6c7b11ad 100644
--- a/datafusion/sqllogictest/test_files/aggregate.slt
+++ b/datafusion/sqllogictest/test_files/aggregate.slt
@@ -725,10 +725,6 @@ SELECT c2, var_samp(c12) FROM aggregate_test_100 WHERE c12 
> 0.90 GROUP BY c2 OR
 4 NULL
 5 0.000269544643
 
-# Use PostgresSQL dialect
-statement ok
-set datafusion.sql_parser.dialect = 'Postgres';
-
 # csv_query_stddev_12
 query IR
 SELECT c2, var_samp(c12) FILTER (WHERE c12 > 0.95) FROM aggregate_test_100 
GROUP BY c2 ORDER BY c2
@@ -764,10 +760,6 @@ WITH s AS (
 statement ok
 DROP TABLE t
 
-# Restore the default dialect
-statement ok
-set datafusion.sql_parser.dialect = 'Generic';
-
 # csv_query_stddev_13
 query IR
 SELECT c2, var_samp(CASE WHEN c12 > 0.90 THEN c12 ELSE null END) FROM 
aggregate_test_100 GROUP BY c2 ORDER BY c2
@@ -5197,10 +5189,6 @@ select c2, count(DISTINCT cast(c1 AS DECIMAL(10, 2))) 
from d_table GROUP BY c2 O
 A 2
 B 2
 
-# Use PostgresSQL dialect
-statement ok
-set datafusion.sql_parser.dialect = 'Postgres';
-
 # Creating the table
 statement ok
 CREATE TABLE test_table (c1 INT, c2 INT, c3 INT)
@@ -5330,10 +5318,6 @@ select c3, count(c2), avg(c2), sum(c2), min(c2), 
max(c2), count(c4), sum(c4) fro
 700.1 2 15.15 30.3 10.1 20.2 0 NULL
 NULL 1 10.1 10.1 10.1 10.1 0 NULL
 
-# Restore the default dialect
-statement ok
-set datafusion.sql_parser.dialect = 'Generic';
-
 ## Multiple distinct aggregates and dictionaries
 statement ok
 create table dict_test as values (1, arrow_cast('foo', 'Dictionary(Int32, 
Utf8)')), (1, arrow_cast('foo', 'Dictionary(Int32, Utf8)')), (2, 
arrow_cast('bar', 'Dictionary(Int32, Utf8)')), (1, arrow_cast('bar', 
'Dictionary(Int32, Utf8)'));
diff --git a/datafusion/sqllogictest/test_files/aggregate_skip_partial.slt 
b/datafusion/sqllogictest/test_files/aggregate_skip_partial.slt
index 8755918cd1..5dcb72b705 100644
--- a/datafusion/sqllogictest/test_files/aggregate_skip_partial.slt
+++ b/datafusion/sqllogictest/test_files/aggregate_skip_partial.slt
@@ -69,9 +69,6 @@ set datafusion.execution.target_partitions = 2;
 statement ok
 set datafusion.execution.batch_size = 1;
 
-statement ok
-set datafusion.sql_parser.dialect = 'Postgres';
-
 # Grouping by unique fields allows to check all accumulators
 query ITIIII
 SELECT c5, c1,
@@ -420,10 +417,6 @@ c true false NULL
 d NULL false NULL
 e true false NULL
 
-# Enabling PG dialect for filtered aggregates tests
-statement ok
-set datafusion.sql_parser.dialect = 'Postgres';
-
 # Test count with filter
 query III
 SELECT
diff --git a/datafusion/sqllogictest/test_files/create_function.slt 
b/datafusion/sqllogictest/test_files/create_function.slt
index 4f0c53c36c..4e82c0866e 100644
--- a/datafusion/sqllogictest/test_files/create_function.slt
+++ b/datafusion/sqllogictest/test_files/create_function.slt
@@ -21,11 +21,6 @@
 ## Note that DataFusion provides a pluggable system for creating functions
 ## but has no built in support for doing so.
 
-# Use PostgresSQL dialect (until we upgrade to sqlparser 0.44, where CREATE 
FUNCTION)
-# is supported in the Generic dialect (the default)
-statement ok
-set datafusion.sql_parser.dialect = 'Postgres';
-
 # Create function will fail unless a user supplied function factory is supplied
 statement error DataFusion error: Invalid or Unsupported Configuration: 
Function factory has not been configured
 CREATE FUNCTION foo (DOUBLE) RETURNS DOUBLE RETURN $1 + $2;
diff --git a/datafusion/sqllogictest/test_files/expr.slt 
b/datafusion/sqllogictest/test_files/expr.slt
index eeea3cd39a..87345b833e 100644
--- a/datafusion/sqllogictest/test_files/expr.slt
+++ b/datafusion/sqllogictest/test_files/expr.slt
@@ -2079,9 +2079,6 @@ host1 1.1 101
 host2 2.2 202
 host3 3.3 303
 
-statement ok
-set datafusion.sql_parser.dialect = 'Postgres';
-
 statement ok
 create table t (a float) as values (1), (2), (3);
 
@@ -2101,9 +2098,6 @@ physical_plan
 statement ok
 drop table t;
 
-statement ok
-set datafusion.sql_parser.dialect = 'Generic';
-
 # test between expression with null
 query I
 select 1 where null between null and null;
diff --git a/datafusion/sqllogictest/test_files/group_by.slt 
b/datafusion/sqllogictest/test_files/group_by.slt
index 1b5ea3df2c..b72f73d446 100644
--- a/datafusion/sqllogictest/test_files/group_by.slt
+++ b/datafusion/sqllogictest/test_files/group_by.slt
@@ -4475,10 +4475,6 @@ physical_plan
 12)----------------------RepartitionExec: partitioning=RoundRobinBatch(8), 
input_partitions=1
 13)------------------------DataSourceExec: file_groups={1 group: 
[[WORKSPACE_ROOT/testing/data/csv/aggregate_test_100.csv]]}, projection=[c1, 
c2, c3, c4], file_type=csv, has_header=true
 
-# Use PostgreSQL dialect
-statement ok
-set datafusion.sql_parser.dialect = 'Postgres';
-
 query II
 SELECT c2, count(distinct c3) FILTER (WHERE c1 != 'a') FROM aggregate_test_100 
GROUP BY c2 ORDER BY c2;
 ----
@@ -4497,10 +4493,6 @@ SELECT c2, count(distinct c3) FILTER (WHERE c1 != 'a'), 
count(c5) FILTER (WHERE
 4 19 18
 5 11 9
 
-# Restore the default dialect
-statement ok
-set datafusion.sql_parser.dialect = 'Generic';
-
 statement ok
 drop table aggregate_test_100;
 


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to