A cosmetic suggest for 4.2.7. Aggregate Expressions

2018-07-03 Thread PG Doc comments form
The following documentation comment has been logged on the website:

Page: https://www.postgresql.org/docs/10/static/sql-expressions.html
Description:

Documentation for PostgreSQL 10.4
4.2.7. Aggregate Expressions
...
aggregate_name (expression [ , ... ] [ order_by_clause ] ) [ FILTER ( WHERE
filter_clause ) ]
aggregate_name (ALL expression [ , ... ] [ order_by_clause ] ) [ FILTER (
WHERE filter_clause ) ]
...
The first form of aggregate expression invokes the aggregate once for each
input row. The second form is the same as the first, since ALL is the
default.
...

Since both definitions are the same will be better rewrite to
aggregate_name ([ALL ]expression [ , ... ] [ order_by_clause ] ) [ FILTER (
WHERE filter_clause ) ]
to emphasize this.


Re: Dead link to hp docs

2018-07-03 Thread Magnus Hagander
On Tue, Jul 3, 2018 at 7:37 AM, Tom Lane  wrote:

> Alvaro Herrera  writes:
> > On 2018-Jul-02, Michael Paquier wrote:
> >> Here are all the broken links in the source tree:
> >>
> >> doc/src/sgml/libpq.sgml:
> >> Wrong:   http://h71000.www7.hp.com/doc/83final/ba554_90007/ch04.html
> >> Correct: http://h41379.www4.hpe.com/doc/83final/ba554_90007/ch04.html
> >>
> >> doc/src/sgml/runtime.sgml:
> >> Wrong: http://h71000.www7.hp.com/doc/83final/ba554_90007/ch04s02.html
> >> Correct: http://h41379.www4.hpe.com/doc/83final/ba554_90007/
> ch04s02.html
>
> > I think HP break their site on a regular basis.
>
> Well, more to the point, it's obvious from the very form of these links
> that HP don't intend them to be stable externally-referenceable URLs.
> I remember being unhappy from the beginning that Bruce would try to link
> to them in our docs, and I think we're seeing now that indeed they're
> not trustworthy permanent URLs.
>
> > Maybe we should be
> > looking for some more stable source of similar information?
>
> Exactly.
>

+. I've been thinking that for a while. And it's not the first time
they changed things on us.


-- 
 Magnus Hagander
 Me: https://www.hagander.net/ 
 Work: https://www.redpill-linpro.com/