Skip to content

Comments

[INLONG-7149][Manager] Replace tableName in ClickHouseLoadNode with databaseName.tableName#7150

Merged
dockerzhang merged 1 commit intoapache:masterfrom
fuweng11:INLONG-7149
Jan 4, 2023
Merged

[INLONG-7149][Manager] Replace tableName in ClickHouseLoadNode with databaseName.tableName#7150
dockerzhang merged 1 commit intoapache:masterfrom
fuweng11:INLONG-7149

Conversation

@fuweng11
Copy link
Contributor

@fuweng11 fuweng11 commented Jan 4, 2023

Prepare a Pull Request

Motivation

Fix the problem of tableName parameter in ClickHouseLoadNode is incorrect.

Modifications

Replace tableName in ClickHouseLoadNode with databaseName.tableName.

Copy link
Contributor

@vernedeng vernedeng left a comment

Choose a reason for hiding this comment

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

lgtm

@gong
Copy link
Contributor

gong commented Jan 4, 2023

ref to #6595

@dockerzhang dockerzhang merged commit ad28d55 into apache:master Jan 4, 2023
dockerzhang pushed a commit that referenced this pull request Jan 5, 2023
liaorui pushed a commit to liaorui/inlong that referenced this pull request Feb 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug][Manager] The tableName parameter in ClickHouseLoadNode is incorrect

4 participants