Skip to content

Commit 993674e

Browse files
committed
Fix incorrect markup in documentation of window frame clauses.
You're required to write either RANGE or ROWS to start a frame clause, but the documentation incorrectly implied this is optional. Noted by David Johnston.
1 parent 71b3689 commit 993674e

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

doc/src/sgml/ref/select.sgml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -651,8 +651,8 @@ WINDOW <replaceable class="parameter">window_name</replaceable> AS ( <replaceabl
651651
The <replaceable class="parameter">frame_clause</> can be one of
652652

653653
<synopsis>
654-
[ RANGE | ROWS ] <replaceable>frame_start</>
655-
[ RANGE | ROWS ] BETWEEN <replaceable>frame_start</> AND <replaceable>frame_end</>
654+
{ RANGE | ROWS } <replaceable>frame_start</>
655+
{ RANGE | ROWS } BETWEEN <replaceable>frame_start</> AND <replaceable>frame_end</>
656656
</synopsis>
657657

658658
where <replaceable>frame_start</> and <replaceable>frame_end</> can be

doc/src/sgml/syntax.sgml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1725,8 +1725,8 @@ SELECT string_agg(a ORDER BY a, ',') FROM table; -- incorrect
17251725
and the optional <replaceable class="parameter">frame_clause</replaceable>
17261726
can be one of
17271727
<synopsis>
1728-
[ RANGE | ROWS ] <replaceable>frame_start</>
1729-
[ RANGE | ROWS ] BETWEEN <replaceable>frame_start</> AND <replaceable>frame_end</>
1728+
{ RANGE | ROWS } <replaceable>frame_start</>
1729+
{ RANGE | ROWS } BETWEEN <replaceable>frame_start</> AND <replaceable>frame_end</>
17301730
</synopsis>
17311731
where <replaceable>frame_start</> and <replaceable>frame_end</> can be
17321732
one of

0 commit comments

Comments
 (0)