Skip to content

[INLONG-7513][DataProxy] Delete duplicate definitions#7514

Merged
dockerzhang merged 5 commits intoapache:masterfrom
lby1119:duplicate_definition
Mar 6, 2023
Merged

[INLONG-7513][DataProxy] Delete duplicate definitions#7514
dockerzhang merged 5 commits intoapache:masterfrom
lby1119:duplicate_definition

Conversation

@lby1119
Copy link
Copy Markdown
Contributor

@lby1119 lby1119 commented Mar 3, 2023

Motivation

There are duplicate definitions here, so delete the duplicate part.

Modifications

Delete the duplicate definition part.

Verifying this change

(Please pick either of the following options)

  • This change is a trivial rework/code cleanup without any test coverage.

@healchow healchow changed the title [INLONG-7513][DataProxy]Delete duplicate definitions [INLONG-7513][DataProxy] Delete duplicate definitions Mar 4, 2023
@healchow
Copy link
Copy Markdown
Member

healchow commented Mar 4, 2023

Thanks for your contribution 🎉

Copy link
Copy Markdown
Member

@healchow healchow left a comment

Choose a reason for hiding this comment

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

LGTM

lby1119 and others added 3 commits March 6, 2023 14:19
…ong/dataproxy/sink/mq/pulsar/PulsarHandler.java

Co-authored-by: healchow <healchow@gmail.com>
Copy link
Copy Markdown
Member

@healchow healchow left a comment

Choose a reason for hiding this comment

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

LGTM

@dockerzhang dockerzhang merged commit 44effbf into apache:master Mar 6, 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.

[Improve][DataProxy] Delete duplicate definitions

3 participants