Skip to content

Commit 5076912

Browse files
committed
fix flaky explain_analyze.slt
1 parent 65f337d commit 5076912

1 file changed

Lines changed: 10 additions & 4 deletions

File tree

datafusion/sqllogictest/test_files/explain_analyze.slt

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -100,10 +100,16 @@ CREATE EXTERNAL TABLE skew_parquet
100100
STORED AS PARQUET
101101
LOCATION 'test_files/scratch/explain_analyze/output_rows_skew';
102102

103-
query TT
104-
EXPLAIN ANALYZE SELECT * FROM skew_parquet;
105-
----
106-
Plan with Metrics DataSourceExec: <slt:ignore>output_rows_skew=84.31%<slt:ignore>
103+
# Skipped:
104+
# After https://github.com/apache/datafusion/pull/21351, the Parquet scan order
105+
# is no longer deterministic, so this metric cannot be reliably tested.
106+
# Coverage can be restored by implementing this metric in other executors,
107+
# such as the memory table.
108+
#
109+
# query TT
110+
# EXPLAIN ANALYZE SELECT * FROM skew_parquet;
111+
# ----
112+
# Plan with Metrics DataSourceExec: <slt:ignore>output_rows_skew=84.31%<slt:ignore>
107113

108114
# All partition's output_rows: [0]
109115
statement ok

0 commit comments

Comments
 (0)