pgsql: Still more code review for single-page hash vacuuming.

From: Robert Haas <rhaas(at)postgresql(dot)org>
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Still more code review for single-page hash vacuuming.
Date: 2017-03-27 16:54:59
Message-ID: E1csXv1-00013G-AS@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Still more code review for single-page hash vacuuming.

Most seriously, fix use of incorrect block ID, per a report from
Jeff Janes that it causes a crash and a diagnosis from Amit Kapila.

Improve consistency between the hash and btree versions of this
code by adding back a PANIC that btree has, and by registering
data in the xlog record in the same way, per complaints from
Jeff Janes and Amit Kapila.

Tidy up some minor cosmetic points, per complaints from Amit
Kapila.

Patch by Ashutosh Sharma, reviewed by Amit Kapila, and tested by
Jeff Janes.

Discussion: http://postgr.es/m/CAMkU=1w-9Qe=Ff1o6bSaXpNO9wqpo7_9GL8_CVhw4BoVVHasqg@mail.gmail.com

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/c4c51541e22bf7f2da8ecf6986271687b0d7a90e

Modified Files
--------------
src/backend/access/hash/hash_xlog.c | 30 +++++++++++++-----------------
src/backend/access/hash/hashinsert.c | 17 +++++++++--------
src/backend/access/rmgrdesc/hashdesc.c | 2 +-
src/include/access/hash_xlog.h | 6 ++++--
4 files changed, 27 insertions(+), 28 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Peter Eisentraut 2017-03-27 17:34:31 pgsql: doc: Improve rendering of notes/cautions using XSL-FO
Previous Message Teodor Sigaev 2017-03-27 16:33:50 pgsql: Fsync directory after creating or unlinking file.