Skip to content

docs: correct pg_dump connection environment variables on master - #345

Open
Wang1rrr wants to merge 1 commit into
IvorySQL:masterfrom
Wang1rrr:docs/pg-dump-connection-env-master
Open

Wang1rrr wants to merge 1 commit into
IvorySQL:masterfrom
Wang1rrr:docs/pg-dump-connection-env-master

Conversation

@Wang1rrr

Copy link
Copy Markdown
Contributor

Summary

Apply the same correction as #343 (targeting v5.6) to master, as
requested by jiaoshuntian.
Related issue: #342.

On master, the corresponding pages are the English and Chinese
modules/ROOT/pages/master/operation_guide.adoc files. Correct HOST and
PORT to PGHOST and PGPORT in their pg_dump connection paragraphs.

Why this correction is needed

libpq reads PGHOST and PGPORT for the default connection host and port.
Following the current documentation and setting HOST or PORT does not
configure those defaults, so pg_dump may attempt the wrong endpoint or fail
to connect.

This keeps the two maintained branches and both translations aligned.

Verification

Use PGHOST and PGPORT in the English and Chinese operation guides, matching
the environment variables recognized by libpq.

Apply the correction from IvorySQL#343 to master as requested by the maintainer.

Refs IvorySQL#342
Copilot AI balanced review requested due to automatic review settings October 20, 2026 07:53

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions
github-actions Bot temporarily deployed to pull request October 20, 2026 07:56 Inactive
@github-actions

Copy link
Copy Markdown

🚀 IvorySQL-Docs Preview Ready

This branch was previously deployed

1 inactive deployment
pull request — 4149c2df Deployed Oct 20, 2026 by github-actions[bot]
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.

2 participants