Skip to content

pg_dump: minor bug fixes - #71

Merged
Smyatkin-Maxim merged 1 commit into
REL_2_STABLEfrom
pg-dump-fe-fixes
Sep 14, 2026
Merged

pg_dump: minor bug fixes#71
Smyatkin-Maxim merged 1 commit into
REL_2_STABLEfrom
pg-dump-fe-fixes

Conversation

@Smyatkin-Maxim

Copy link
Copy Markdown

A bunch of small but important fixes for pg_dump bugs (introduced in earlier sync with upstream):

  • Keep the tgisinternal result-column name in PostgreSQL 11+ trigger
    queries, matching the consumer and avoiding an invalid column lookup.
  • Add the missing comma after the legacy aggmfinalmodify fallback.
  • Read attencoding from result row r instead of table-local index j.

- Keep the tgisinternal result-column name in PostgreSQL 11+ trigger
queries, matching the consumer and avoiding an invalid column lookup.
- Add the missing comma after the legacy aggmfinalmodify fallback.
- Read attencoding from result row r instead of table-local index j.

@leborchuk leborchuk left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Smyatkin-Maxim
Smyatkin-Maxim merged commit 20405bd into REL_2_STABLE Sep 14, 2026
190 of 199 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants