[COMMITTERS] pgsql: Fix regression test parallel-make hazard.

2016-08-12 Thread Tom Lane
Fix regression test parallel-make hazard. Back-patch 9.4-era commit 384f933046dc9e9a2b416f5f7b3be30b93587c63 into the previous branches. Although that was only advertised as repairing a problem with missed header-file dependencies, it turns out to also be important for parallel make safety. The

[COMMITTERS] pgsql: Fix regression test parallel-make hazard.

2016-08-12 Thread Tom Lane
Fix regression test parallel-make hazard. Back-patch 9.4-era commit 384f933046dc9e9a2b416f5f7b3be30b93587c63 into the previous branches. Although that was only advertised as repairing a problem with missed header-file dependencies, it turns out to also be important for parallel make safety. The

[COMMITTERS] pgsql: Fix regression test parallel-make hazard.

2016-08-12 Thread Tom Lane
Fix regression test parallel-make hazard. Back-patch 9.4-era commit 384f933046dc9e9a2b416f5f7b3be30b93587c63 into the previous branches. Although that was only advertised as repairing a problem with missed header-file dependencies, it turns out to also be important for parallel make safety. The

[COMMITTERS] pgsql: Doc: clarify that DROP ... CASCADE is recursive.

2016-08-12 Thread Tom Lane
Doc: clarify that DROP ... CASCADE is recursive. Apparently that's not obvious to everybody, so let's belabor the point. In passing, document that DROP POLICY has CASCADE/RESTRICT options (which it does, per gram.y) but they do nothing (I assume, anyway). Also update some long-obsolete commentar

[COMMITTERS] pgsql: Fix inappropriate printing of never-measured times in EXPLAIN.

2016-08-12 Thread Tom Lane
Fix inappropriate printing of never-measured times in EXPLAIN. EXPLAIN (ANALYZE, TIMING OFF) would print an elapsed time of zero for a trigger function, because no measurement has been taken but it printed the field anyway. This isn't what EXPLAIN does elsewhere, so suppress it. In the same vein

[COMMITTERS] pgsql: Fix inappropriate printing of never-measured times in EXPLAIN.

2016-08-12 Thread Tom Lane
Fix inappropriate printing of never-measured times in EXPLAIN. EXPLAIN (ANALYZE, TIMING OFF) would print an elapsed time of zero for a trigger function, because no measurement has been taken but it printed the field anyway. This isn't what EXPLAIN does elsewhere, so suppress it. In the same vein

[COMMITTERS] pgsql: Fix inappropriate printing of never-measured times in EXPLAIN.

2016-08-12 Thread Tom Lane
Fix inappropriate printing of never-measured times in EXPLAIN. EXPLAIN (ANALYZE, TIMING OFF) would print an elapsed time of zero for a trigger function, because no measurement has been taken but it printed the field anyway. This isn't what EXPLAIN does elsewhere, so suppress it. In the same vein

[COMMITTERS] pgsql: Fix inappropriate printing of never-measured times in EXPLAIN.

2016-08-12 Thread Tom Lane
Fix inappropriate printing of never-measured times in EXPLAIN. EXPLAIN (ANALYZE, TIMING OFF) would print an elapsed time of zero for a trigger function, because no measurement has been taken but it printed the field anyway. This isn't what EXPLAIN does elsewhere, so suppress it. In the same vein

[COMMITTERS] pgsql: Fix inappropriate printing of never-measured times in EXPLAIN.

2016-08-12 Thread Tom Lane
Fix inappropriate printing of never-measured times in EXPLAIN. EXPLAIN (ANALYZE, TIMING OFF) would print an elapsed time of zero for a trigger function, because no measurement has been taken but it printed the field anyway. This isn't what EXPLAIN does elsewhere, so suppress it. In the same vein

[COMMITTERS] pgsql: Code cleanup in SyncRepWaitForLSN()

2016-08-12 Thread Simon Riggs
Code cleanup in SyncRepWaitForLSN() Commit 14e8803f1 removed LWLocks when accessing MyProc->syncRepState but didn't clean up the surrounding code and comments. Cleanup and backpatch to 9.5, to keep code similar. Julien Rouhaud, improved by suggestion from Michael Paquier, implemented trivially b

[COMMITTERS] pgsql: Code cleanup in SyncRepWaitForLSN()

2016-08-12 Thread Simon Riggs
Code cleanup in SyncRepWaitForLSN() Commit 14e8803f1 removed LWLocks when accessing MyProc->syncRepState but didn't clean up the surrounding code and comments. Cleanup and backpatch to 9.5, to keep code similar. Julien Rouhaud, improved by suggestion from Michael Paquier, implemented trivially b

[COMMITTERS] pgsql: Correct TABLESAMPLE docs

2016-08-12 Thread Simon Riggs
Correct TABLESAMPLE docs Original wording was correct but not the intended meaning. Reported by Patrik Wenger Branch -- REL9_5_STABLE Details --- http://git.postgresql.org/pg/commitdiff/270c29f125f5b5d2a199fb98fdf9c289640093a3 Modified Files -- doc/src/sgml/ref/select.sgml

[COMMITTERS] pgsql: Correct TABLESAMPLE docs

2016-08-12 Thread Simon Riggs
Correct TABLESAMPLE docs Original wording was correct but not the intended meaning. Reported by Patrik Wenger Branch -- master Details --- http://git.postgresql.org/pg/commitdiff/6e75559ea9cb0bc8fd07543567cd02d4ec85ca20 Modified Files -- doc/src/sgml/ref/select.sgml | 2 +-