[COMMITTERS] pgsql: In pgbench logging, avoid assuming that instr_times match Unix t

2017-01-02 Thread Tom Lane
In pgbench logging, avoid assuming that instr_times match Unix timestamps. For aggregated logging, pg_bench supposed that printing the integer part of INSTR_TIME_GET_DOUBLE() would produce a Unix timestamp. That was already broken on Windows, and it's about to get broken on most other platforms a

[COMMITTERS] pgsql: Use clock_gettime(), if available, in instr_time measurements.

2017-01-02 Thread Tom Lane
Use clock_gettime(), if available, in instr_time measurements. The advantage of clock_gettime() is that the API allows the result to be precise to nanoseconds, not just microseconds as in gettimeofday(). Now that it's routinely possible to do tens of plan node executions in 1us, we really need mor

[COMMITTERS] pgsql: Silence Bison deprecation warnings

2017-01-02 Thread Joe Conway
Silence Bison deprecation warnings Bison >=3.0 issues warnings about %name-prefix="base_yy" instead of the now preferred %name-prefix "base_yy" but the latter doesn't work with Bison 2.3 or less. So for now we silence the deprecation warnings. Back-patch to 9.2 and 9.3 -- the newer b

[COMMITTERS] pgsql: Silence Bison deprecation warnings

2017-01-02 Thread Joe Conway
Silence Bison deprecation warnings Bison >=3.0 issues warnings about %name-prefix="base_yy" instead of the now preferred %name-prefix "base_yy" but the latter doesn't work with Bison 2.3 or less. So for now we silence the deprecation warnings. Back-patch to 9.2 and 9.3 -- the newer b

[COMMITTERS] pgsql: Silence compiler warnings

2017-01-02 Thread Joe Conway
Silence compiler warnings In GetCachedPlan(), initialize 'plan' to silence a compiler warning, but also add an Assert() to make sure we don't ever actually fall through with 'plan' still being set to NULL, since we are about to dereference it. Back-patch back to 9.2. Author: Stephen Frost Discus

[COMMITTERS] pgsql: Silence compiler warnings

2017-01-02 Thread Joe Conway
Silence compiler warnings In GetCachedPlan(), initialize 'plan' to silence a compiler warning, but also add an Assert() to make sure we don't ever actually fall through with 'plan' still being set to NULL, since we are about to dereference it. Back-patch back to 9.2. Author: Stephen Frost Discus

[COMMITTERS] pgsql: Silence compiler warnings

2017-01-02 Thread Joe Conway
Silence compiler warnings In GetCachedPlan(), initialize 'plan' to silence a compiler warning, but also add an Assert() to make sure we don't ever actually fall through with 'plan' still being set to NULL, since we are about to dereference it. Back-patch back to 9.2. Author: Stephen Frost Discus

[COMMITTERS] pgsql: Silence compiler warnings

2017-01-02 Thread Joe Conway
Silence compiler warnings In GetCachedPlan(), initialize 'plan' to silence a compiler warning, but also add an Assert() to make sure we don't ever actually fall through with 'plan' still being set to NULL, since we are about to dereference it. Back-patch back to 9.2. Author: Stephen Frost Discus

[COMMITTERS] pgsql: ilence compiler warnings

2017-01-02 Thread Joe Conway
ilence compiler warnings In GetCachedPlan(), initialize 'plan' to silence a compiler warning, but also add an Assert() to make sure we don't ever actually fall through with 'plan' still being set to NULL, since we are about to dereference it. Back-patch back to 9.2. Author: Stephen Frost Discuss

[COMMITTERS] pgsql: Silence compiler warnings

2017-01-02 Thread Joe Conway
Silence compiler warnings Rearrange a bit of code to ensure that 'mode' in LWLockRelease is obviously always set, which seems a bit cleaner and avoids a compiler warning (thanks to Robert for the suggestion!). Back-patch back to 9.5 where the warning is first seen. Author: Stephen Frost Discussi

[COMMITTERS] pgsql: Silence compiler warnings

2017-01-02 Thread Joe Conway
Silence compiler warnings Rearrange a bit of code to ensure that 'mode' in LWLockRelease is obviously always set, which seems a bit cleaner and avoids a compiler warning (thanks to Robert for the suggestion!). Back-patch back to 9.5 where the warning is first seen. Author: Stephen Frost Discussi

[COMMITTERS] pgsql: Allow SSL configuration to be updated at SIGHUP.

2017-01-02 Thread Tom Lane
Allow SSL configuration to be updated at SIGHUP. It is no longer necessary to restart the server to enable, disable, or reconfigure SSL. Instead, we just create a new SSL_CTX struct (by re-reading all relevant files) whenever we get SIGHUP. Testing shows that this is fast enough that it shouldn'