Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[INLONG-7353][Agent] Fix error of PostgreSQL connector #7354

Merged
merged 1 commit into from Feb 12, 2023
Merged

[INLONG-7353][Agent] Fix error of PostgreSQL connector #7354

merged 1 commit into from Feb 12, 2023

Conversation

haifxu
Copy link
Contributor

@haifxu haifxu commented Feb 10, 2023

Prepare a Pull Request

Motivation

  1. AgentMetricItem is repeatedly defined and not initialized, just use the one in the abstract parent class AbstractReader.
  2. If deploying multiple instances of the Debezium Postgres connector, must use distinct replication slot names.

Modifications

  1. Remove redundant definitions.
  2. Set the slot name.

Verifying this change

image

@dockerzhang dockerzhang merged commit 98b8f44 into apache:master Feb 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dockerzhang dockerzhang dockerzhang approved these changes

@healchow healchow healchow approved these changes

Assignees
No one assigned
Labels
component/agent
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Bug][Agent] The connector of PostgreSQL is closed
3 participants
@haifxu @dockerzhang @healchow