pgsql: Adopt the GNU convention for handling tar-archive members exceed

Поиск
Список
Период
Сортировка
От Tom Lane
Тема pgsql: Adopt the GNU convention for handling tar-archive members exceed
Дата
Msg-id E1a0JMh-0002cs-NJ@gemulon.postgresql.org
обсуждение исходный текст
Список pgsql-committers
Adopt the GNU convention for handling tar-archive members exceeding 8GB.

The POSIX standard for tar headers requires archive member sizes to be
printed in octal with at most 11 digits, limiting the representable file
size to 8GB.  However, GNU tar and apparently most other modern tars
support a convention in which oversized values can be stored in base-256,
allowing any practical file to be a tar member.  Adopt this convention
to remove two limitations:
* pg_dump with -Ft output format failed if the contents of any one table
exceeded 8GB.
* pg_basebackup failed if the data directory contained any file exceeding
8GB.  (This would be a fatal problem for installations configured with a
table segment size of 8GB or more, and it has also been seen to fail when
large core dump files exist in the data directory.)

File sizes under 8GB are still printed in octal, so that no compatibility
issues are created except in cases that would have failed entirely before.

In addition, this patch fixes several bugs in the same area:

* In 9.3 and later, we'd defined tarCreateHeader's file-size argument as
size_t, which meant that on 32-bit machines it would write a corrupt tar
header for file sizes between 4GB and 8GB, even though no error was raised.
This broke both "pg_dump -Ft" and pg_basebackup for such cases.

* pg_restore from a tar archive would fail on tables of size between 4GB
and 8GB, on machines where either "size_t" or "unsigned long" is 32 bits.
This happened even with an archive file not affected by the previous bug.

* pg_basebackup would fail if there were files of size between 4GB and 8GB,
even on 64-bit machines.

* In 9.3 and later, "pg_basebackup -Ft" failed entirely, for any file size,
on 64-bit big-endian machines.

In view of these potential data-loss bugs, back-patch to all supported
branches, even though removal of the documented 8GB limit might otherwise
be considered a new feature rather than a bug fix.

Branch
------
REL9_2_STABLE

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

Modified Files
--------------
doc/src/sgml/ref/pg_dump.sgml         |   19 +--
src/backend/replication/basebackup.c  |  146 +----------------------
src/bin/pg_basebackup/pg_basebackup.c |   17 +--
src/bin/pg_dump/pg_backup_tar.c       |  149 ++++-------------------
src/include/pgtar.h                   |   18 +++
src/port/Makefile                     |    2 +-
src/port/tar.c                        |  209 +++++++++++++++++++++++++++++++++
7 files changed, 267 insertions(+), 293 deletions(-)


В списке pgsql-committers по дате отправления:

Предыдущее
От: Tom Lane
Дата:
Сообщение: pgsql: Adopt the GNU convention for handling tar-archive members exceed
Следующее
От: Tom Lane
Дата:
Сообщение: pgsql: Adopt the GNU convention for handling tar-archive members exceed