From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | pgsql-committers(at)postgresql(dot)org |
Subject: | pgsql: Fix pg_restore's processing of old-style BLOB COMMENTS data. |
Date: | 2014-06-13 00:15:13 |
Message-ID: | E1WvF9F-0001G2-Fh@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Fix pg_restore's processing of old-style BLOB COMMENTS data.
Prior to 9.0, pg_dump handled comments on large objects by dumping a bunch
of COMMENT commands into a single BLOB COMMENTS archive object. With
sufficiently many such comments, some of the commands would likely get
split across bufferloads when restoring, causing failures in
direct-to-database restores (though no problem would be evident in text
output). This is the same type of issue we have with table data dumped as
INSERT commands, and it can be fixed in the same way, by using a mini SQL
lexer to figure out where the command boundaries are. Fortunately, the
COMMENT commands are no more complex to lex than INSERTs, so we can just
re-use the existing lexer for INSERTs.
Per bug #10611 from Jacek Zalewski. Back-patch to all active branches.
Branch
------
REL9_1_STABLE
Details
-------
http://git.postgresql.org/pg/commitdiff/294a489855c6080197d40673fe592d6b494db6d5
Modified Files
--------------
src/bin/pg_dump/pg_backup_archiver.c | 6 ++++++
src/bin/pg_dump/pg_backup_db.c | 12 +++++++++---
src/bin/pg_dump/pg_dump.c | 2 +-
3 files changed, 16 insertions(+), 4 deletions(-)
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2014-06-13 04:03:05 | pgsql: Improve predtest.c's ability to reason about operator expression |
Previous Message | Tom Lane | 2014-06-12 22:59:19 | pgsql: Improve tuplestore's error messages for I/O failures. |