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

From Robert Haas
Subject pgsql: Still more code review for single-page hash vacuuming.
Date
Msg-id E1csXv1-00013G-AS@gemulon.postgresql.org
Whole thread Raw
List 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(-)


pgsql-committers by date:

Previous
From: Teodor Sigaev
Date:
Subject: pgsql: Fsync directory after creating or unlinking file.
Next
From: Peter Eisentraut
Date:
Subject: pgsql: doc: Improve rendering of notes/cautions using XSL-FO