Marginal editorial improvements for array_agg patch documentation.
authorTom Lane <tgl@sss.pgh.pa.us>
Thu, 13 Nov 2008 23:01:09 +0000 (23:01 +0000)
committerTom Lane <tgl@sss.pgh.pa.us>
Thu, 13 Nov 2008 23:01:09 +0000 (23:01 +0000)
doc/src/sgml/func.sgml

index c1ecd9861fab764ec6f105fae196703003e02916..73a794c7f6dfff48aebc9cf9509de432adccb54f 100644 (file)
@@ -8387,8 +8387,8 @@ SELECT xmlagg(x) FROM test;
     </para>
 
     <para>
-     The influence the order of the concatenation, something like the
-     following approach to sort the input values can be used:
+     To determine the order of the concatenation, something like the
+     following approach can be used:
 
 <screen><![CDATA[
 SELECT xmlagg(x) FROM (SELECT * FROM test ORDER BY y DESC) AS tab;
@@ -9772,19 +9772,19 @@ SELECT count(*) FROM sometable;
    and <function>xmlagg</function>, as well as similar user-defined
    aggregate functions, produce meaningfully different result values
    depending on the order of the input values.  In the current
-   implementation, the order of the concatenation is in principle
-   undefined.  Making the input values to be sorted in some other way
+   implementation, the order of the input is in principle unspecified.
+   Supplying the input values from a sorted subquery
    will usually work, however.  For example:
 
 <screen><![CDATA[
-SELECT xmlagg(x) FROM (SELECT * FROM test ORDER BY y DESC) AS tab;
+SELECT xmlagg(x) FROM (SELECT x FROM test ORDER BY y DESC) AS tab;
 ]]></screen>
 
-   But this approach is not guaranteed to work in all situations, and
-   it is not strictly SQL-conforming.  A future version of PostgreSQL
-   might provide an additional feature to control the order in a
-   better-defined way (<literal>xmlagg(expr ORDER BY expr, expr,
-   ...</literal>).
+   But this syntax is not allowed in the SQL standard, and is
+   not portable to other database systems.  A future version of
+   <productname>PostgreSQL</> might provide an additional feature to control
+   the order in a better-defined way (<literal>xmlagg(expr ORDER BY expr, expr,
+   ...)</literal>).
   </para>
 
   <para>