From 711366ae31d7a553689d914db315e8226e8009bf Mon Sep 17 00:00:00 2001 From: Haibo Yan Date: Mon, 6 Apr 2026 23:50:50 -0700 Subject: [PATCH v3 01/10] Parse and deparse DISTINCT in window aggregate calls Add a windistinct field to WindowFunc to carry the DISTINCT qualifier through parse analysis, node serialization, and ruleutils deparse. Move the rejection from parse time to executor init so that stored rules and view definitions preserve the syntax. Execution still raises FEATURE_NOT_SUPPORTED until later patches add real support. --- src/backend/executor/nodeWindowAgg.c | 9 +++++++++ src/backend/optimizer/util/clauses.c | 1 + src/backend/parser/parse_func.c | 6 ++++-- src/backend/utils/adt/ruleutils.c | 2 ++ src/include/nodes/primnodes.h | 2 ++ src/test/regress/expected/window.out | 23 +++++++++++++++++++++++ src/test/regress/sql/window.sql | 18 ++++++++++++++++++ 7 files changed, 59 insertions(+), 2 deletions(-) diff --git a/src/backend/executor/nodeWindowAgg.c b/src/backend/executor/nodeWindowAgg.c index f1c524d00df..6c516e241ab 100644 --- a/src/backend/executor/nodeWindowAgg.c +++ b/src/backend/executor/nodeWindowAgg.c @@ -2957,6 +2957,15 @@ initialize_peragg(WindowAggState *winstate, WindowFunc *wfunc, int i; ListCell *lc; + /* + * Temporary: reject DISTINCT window aggregates until executor support + * lands. Patch 2 will replace this with actual DISTINCT handling. + */ + if (wfunc->windistinct) + ereport(ERROR, + (errcode(ERRCODE_FEATURE_NOT_SUPPORTED), + errmsg("DISTINCT is not yet implemented for window aggregates"))); + numArguments = list_length(wfunc->args); i = 0; diff --git a/src/backend/optimizer/util/clauses.c b/src/backend/optimizer/util/clauses.c index 01997e22266..3ac51617ca0 100644 --- a/src/backend/optimizer/util/clauses.c +++ b/src/backend/optimizer/util/clauses.c @@ -2819,6 +2819,7 @@ eval_const_expressions_mutator(Node *node, newexpr->winref = expr->winref; newexpr->winstar = expr->winstar; newexpr->winagg = expr->winagg; + newexpr->windistinct = expr->windistinct; newexpr->ignore_nulls = expr->ignore_nulls; newexpr->location = expr->location; diff --git a/src/backend/parser/parse_func.c b/src/backend/parser/parse_func.c index fb306c05112..d0420763b86 100644 --- a/src/backend/parser/parse_func.c +++ b/src/backend/parser/parse_func.c @@ -854,6 +854,7 @@ ParseFuncOrColumn(ParseState *pstate, List *funcname, List *fargs, /* winref will be set by transformWindowFuncCall */ wfunc->winstar = agg_star; wfunc->winagg = (fdresult == FUNCDETAIL_AGGREGATE); + wfunc->windistinct = agg_distinct; wfunc->aggfilter = agg_filter; wfunc->ignore_nulls = ignore_nulls; wfunc->runCondition = NIL; @@ -861,11 +862,12 @@ ParseFuncOrColumn(ParseState *pstate, List *funcname, List *fargs, /* * agg_star is allowed for aggregate functions but distinct isn't + * allowed for non-aggregate window functions. */ - if (agg_distinct) + if (agg_distinct && !wfunc->winagg) ereport(ERROR, (errcode(ERRCODE_FEATURE_NOT_SUPPORTED), - errmsg("DISTINCT is not implemented for window functions"), + errmsg("DISTINCT is not implemented for non-aggregate window functions"), parser_errposition(pstate, location))); /* diff --git a/src/backend/utils/adt/ruleutils.c b/src/backend/utils/adt/ruleutils.c index 88de5c0481c..feafd2df9be 100644 --- a/src/backend/utils/adt/ruleutils.c +++ b/src/backend/utils/adt/ruleutils.c @@ -11659,6 +11659,8 @@ get_windowfunc_expr_helper(WindowFunc *wfunc, deparse_context *context, appendStringInfoChar(buf, '*'); else { + if (wfunc->windistinct) + appendStringInfoString(buf, "DISTINCT "); if (is_json_objectagg) { get_rule_expr((Node *) linitial(wfunc->args), context, false); diff --git a/src/include/nodes/primnodes.h b/src/include/nodes/primnodes.h index bb05aeebee4..c1c25699208 100644 --- a/src/include/nodes/primnodes.h +++ b/src/include/nodes/primnodes.h @@ -606,6 +606,8 @@ typedef struct WindowFunc bool winstar pg_node_attr(query_jumble_ignore); /* is function a simple aggregate? */ bool winagg pg_node_attr(query_jumble_ignore); + /* true if aggregate arguments were marked DISTINCT */ + bool windistinct; /* ignore nulls. One of the Null Treatment options */ int ignore_nulls; /* token location, or -1 if unknown */ diff --git a/src/test/regress/expected/window.out b/src/test/regress/expected/window.out index 90d9f953b81..8d7816c5257 100644 --- a/src/test/regress/expected/window.out +++ b/src/test/regress/expected/window.out @@ -6046,3 +6046,26 @@ SELECT last_value FROM null_treatment_seq; --cleanup DROP TABLE planets CASCADE; NOTICE: drop cascades to view planets_view +-- +-- Test DISTINCT in window aggregates (parse/deparse plumbing only; +-- execution support is not yet implemented) +-- +-- Should parse successfully and round-trip through a view definition +CREATE TEMP VIEW window_distinct_view AS +SELECT count(DISTINCT four) OVER (PARTITION BY ten) AS cnt +FROM tenk1; +SELECT pg_get_viewdef('window_distinct_view') LIKE '%DISTINCT%' AS has_distinct; + has_distinct +-------------- + t +(1 row) + +DROP VIEW window_distinct_view; +-- DISTINCT on a non-aggregate window function is still a parse error +SELECT ntile(DISTINCT 4) OVER () FROM tenk1; -- error +ERROR: DISTINCT is not implemented for non-aggregate window functions +LINE 1: SELECT ntile(DISTINCT 4) OVER () FROM tenk1; + ^ +-- Execution fails with a clear executor-side error +SELECT count(DISTINCT four) OVER (PARTITION BY ten) FROM tenk1; -- error +ERROR: DISTINCT is not yet implemented for window aggregates diff --git a/src/test/regress/sql/window.sql b/src/test/regress/sql/window.sql index 5ac3a486e16..8c00a6e3d89 100644 --- a/src/test/regress/sql/window.sql +++ b/src/test/regress/sql/window.sql @@ -2190,3 +2190,21 @@ SELECT last_value FROM null_treatment_seq; --cleanup DROP TABLE planets CASCADE; + +-- +-- Test DISTINCT in window aggregates (parse/deparse plumbing only; +-- execution support is not yet implemented) +-- + +-- Should parse successfully and round-trip through a view definition +CREATE TEMP VIEW window_distinct_view AS +SELECT count(DISTINCT four) OVER (PARTITION BY ten) AS cnt +FROM tenk1; +SELECT pg_get_viewdef('window_distinct_view') LIKE '%DISTINCT%' AS has_distinct; +DROP VIEW window_distinct_view; + +-- DISTINCT on a non-aggregate window function is still a parse error +SELECT ntile(DISTINCT 4) OVER () FROM tenk1; -- error + +-- Execution fails with a clear executor-side error +SELECT count(DISTINCT four) OVER (PARTITION BY ten) FROM tenk1; -- error -- 2.54.0