Skip to content

Commit 2eb59ad

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 6bd7e30 commit 2eb59ad

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
@@ -623,8 +623,8 @@ WINDOW <replaceable class="parameter">window_name</replaceable> AS ( <replaceabl
623623
The <replaceable class="parameter">frame_clause</> can be one of
624624

625625
<synopsis>
626-
[ RANGE | ROWS ] <replaceable>frame_start</>
627-
[ RANGE | ROWS ] BETWEEN <replaceable>frame_start</> AND <replaceable>frame_end</>
626+
{ RANGE | ROWS } <replaceable>frame_start</>
627+
{ RANGE | ROWS } BETWEEN <replaceable>frame_start</> AND <replaceable>frame_end</>
628628
</synopsis>
629629

630630
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
@@ -1695,8 +1695,8 @@ SELECT string_agg(a ORDER BY a, ',') FROM table; -- incorrect
16951695
and the optional <replaceable class="parameter">frame_clause</replaceable>
16961696
can be one of
16971697
<synopsis>
1698-
[ RANGE | ROWS ] <replaceable>frame_start</>
1699-
[ RANGE | ROWS ] BETWEEN <replaceable>frame_start</> AND <replaceable>frame_end</>
1698+
{ RANGE | ROWS } <replaceable>frame_start</>
1699+
{ RANGE | ROWS } BETWEEN <replaceable>frame_start</> AND <replaceable>frame_end</>
17001700
</synopsis>
17011701
where <replaceable>frame_start</> and <replaceable>frame_end</> can be
17021702
one of

0 commit comments

Comments
 (0)