Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Fix incorrect markup in documentation of window frame clauses.
authorTom Lane <tgl@sss.pgh.pa.us>
Wed, 1 Apr 2015 00:02:40 +0000 (20:02 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Wed, 1 Apr 2015 00:03:50 +0000 (20:03 -0400)
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.

doc/src/sgml/ref/select.sgml
doc/src/sgml/syntax.sgml

index 59664e8891f2b569b181df8c21110c1486cf796a..b1727b9807ab8518f9745a71cdd38d8b17c89096 100644 (file)
@@ -704,8 +704,8 @@ WINDOW <replaceable class="parameter">window_name</replaceable> AS ( <replaceabl
     The <replaceable class="parameter">frame_clause</> can be one of
 
 <synopsis>
-[ RANGE | ROWS ] <replaceable>frame_start</>
-[ RANGE | ROWS ] BETWEEN <replaceable>frame_start</> AND <replaceable>frame_end</>
+{ RANGE | ROWS } <replaceable>frame_start</>
+{ RANGE | ROWS } BETWEEN <replaceable>frame_start</> AND <replaceable>frame_end</>
 </synopsis>
 
     where <replaceable>frame_start</> and <replaceable>frame_end</> can be
index 2e5104526a8d5cf0667262d1799510d33653c213..1dfb38eec31d4548de097df1dd09414b54230d90 100644 (file)
@@ -1725,8 +1725,8 @@ SELECT string_agg(a ORDER BY a, ',') FROM table;  -- incorrect
     and the optional <replaceable class="parameter">frame_clause</replaceable>
     can be one of
 <synopsis>
-[ RANGE | ROWS ] <replaceable>frame_start</>
-[ RANGE | ROWS ] BETWEEN <replaceable>frame_start</> AND <replaceable>frame_end</>
+{ RANGE | ROWS } <replaceable>frame_start</>
+{ RANGE | ROWS } BETWEEN <replaceable>frame_start</> AND <replaceable>frame_end</>
 </synopsis>
     where <replaceable>frame_start</> and <replaceable>frame_end</> can be
     one of