| From: | Zsolt Parragi <zsolt(dot)parragi(at)percona(dot)com> |
|---|---|
| To: | Hannu Krosing <hannuk(at)google(dot)com> |
| Cc: | PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, David Rowley <dgrowleyml(at)gmail(dot)com>, Michael Banck <mbanck(at)gmx(dot)net>, Ashutosh Bapat <ashutosh(dot)bapat(dot)oss(at)gmail(dot)com>, Nathan Bossart <nathandbossart(at)gmail(dot)com> |
| Subject: | Re: Patch: dumping tables data in multiple chunks in pg_dump |
| Date: | 2026-03-30 21:32:22 |
| Message-ID: | CAN4CZFN=iKYAhaki0eC-ADutk-aB3B5jOV26FrTewWouQiD6jQ@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hello!
A simple test causes an assertion failure in my testing, dependency
counting still doesn't seem to work correctly:
pg_restore: >...>/pg_backup_archiver.c:5207: reduce_dependencies:
Assertion `otherte->depCount > 0' failed.
Without assertions it results in data loss.
004_pg_dump_parallel also showcases the issue in my testing.
But simple manual testing also confirms it:
1. create some data
CREATE TABLE tplain (id int UNIQUE);
INSERT INTO tplain SELECT x FROM generate_series(1,1000) x;
2. create a dump
dump with --max-table-segment-pages=2
3. try to restore
restore with --jobs=3
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Sami Imseih | 2026-03-30 21:33:05 | Re: [PATCH] pg_stat_statements: add last_execution_start column |
| Previous Message | Nathan Bossart | 2026-03-30 21:13:32 | Re: remove bits* types |