[COMMITTERS] pgsql: Make "postgres -C guc" print "" not "(null)" for null-valued GUC

2016-06-22 Thread Tom Lane
Make "postgres -C guc" print "" not "(null)" for null-valued GUCs. Commit 0b0baf262 et al made this case print "(null)" on the grounds that that's what happened on platforms that didn't crash. But neither behavior was actually intentional. What we should print is just an empty string, for compat

[COMMITTERS] pgsql: Make "postgres -C guc" print "" not "(null)" for null-valued GUC

2016-06-22 Thread Tom Lane
Make "postgres -C guc" print "" not "(null)" for null-valued GUCs. Commit 0b0baf262 et al made this case print "(null)" on the grounds that that's what happened on platforms that didn't crash. But neither behavior was actually intentional. What we should print is just an empty string, for compat

[COMMITTERS] pgsql: Make "postgres -C guc" print "" not "(null)" for null-valued GUC

2016-06-22 Thread Tom Lane
Make "postgres -C guc" print "" not "(null)" for null-valued GUCs. Commit 0b0baf262 et al made this case print "(null)" on the grounds that that's what happened on platforms that didn't crash. But neither behavior was actually intentional. What we should print is just an empty string, for compat

[COMMITTERS] pgsql: Make "postgres -C guc" print "" not "(null)" for null-valued GUC

2016-06-22 Thread Tom Lane
Make "postgres -C guc" print "" not "(null)" for null-valued GUCs. Commit 0b0baf262 et al made this case print "(null)" on the grounds that that's what happened on platforms that didn't crash. But neither behavior was actually intentional. What we should print is just an empty string, for compat

[COMMITTERS] pgsql: Make "postgres -C guc" print "" not "(null)" for null-valued GUC

2016-06-22 Thread Tom Lane
Make "postgres -C guc" print "" not "(null)" for null-valued GUCs. Commit 0b0baf262 et al made this case print "(null)" on the grounds that that's what happened on platforms that didn't crash. But neither behavior was actually intentional. What we should print is just an empty string, for compat

[COMMITTERS] pgsql: Fix type-safety problem with parallel aggregate serial/deseriali

2016-06-22 Thread Tom Lane
Fix type-safety problem with parallel aggregate serial/deserialization. The original specification for this called for the deserialization function to have signature "deserialize(serialtype) returns transtype", which is a security violation if transtype is INTERNAL (which it always would be in pra

[COMMITTERS] pgsql: Update oidjoins regression test for 9.6.

2016-06-22 Thread Tom Lane
Update oidjoins regression test for 9.6. Looks like we had some more catalog drift recently. Branch -- master Details --- http://git.postgresql.org/pg/commitdiff/63ae052367c350935c3cec3e3c53a1e34a317e96 Modified Files -- src/test/regress/expected/oidjoins.out | 32 ++

[COMMITTERS] pgsql: Improve user-facing documentation for partial/parallel aggregati

2016-06-22 Thread Tom Lane
Improve user-facing documentation for partial/parallel aggregation. Add a section to xaggr.sgml, as we have done in the past for other extensions to the aggregation functionality. Assorted wordsmithing and other minor improvements. David Rowley and Tom Lane Branch -- master Details ---