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.

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

[Feature][Connector-V2][Doris] Add Doris StreamLoad sink connector #3631

Merged
merged 53 commits into from
Dec 14, 2022

Conversation

TaoZex
Copy link
Contributor

@TaoZex TaoZex commented Dec 2, 2022

Purpose of this pull request

#3018

Add Doris StreamLoad sink connector

Doris e2e test:
f126fbe3c1d73edd03c05dfae498d3c

Test json format.Generate 100,000 rows of data using fake source.
a33f4e13237e8b6c40d4027fef43c81

Sink 100000 rows data successfully.
13104c0582aae0755606268fd30a08d

Test csv format.Generate 1000 rows of data using fake source.
1670001509878

Sink 1000 rows data successfully.
1670001584542

Check list

@TaoZex TaoZex marked this pull request as draft December 2, 2022 12:11
@TaoZex TaoZex marked this pull request as ready for review December 2, 2022 17:52
@TaoZex TaoZex closed this Dec 4, 2022
@TaoZex TaoZex reopened this Dec 4, 2022
@TaoZex TaoZex requested a review from TyrantLucifer December 4, 2022 16:22
@TaoZex
Copy link
Contributor Author

TaoZex commented Dec 4, 2022

@EricJoy2048 @Hisoka-X @hailin0 PTAL

hailin0
hailin0 previously approved these changes Dec 8, 2022
Copy link
Member

@hailin0 hailin0 left a comment

Choose a reason for hiding this comment

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

LGTM

hailin0
hailin0 previously approved these changes Dec 8, 2022
hailin0
hailin0 previously approved these changes Dec 8, 2022
@EricJoy2048
Copy link
Member

@TyrantLucifer PTAL.

@TaoZex TaoZex closed this Dec 13, 2022
@TaoZex TaoZex reopened this Dec 13, 2022
Copy link
Member

@Hisoka-X Hisoka-X left a comment

Choose a reason for hiding this comment

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

LGTM

@TaoZex TaoZex requested review from TyrantLucifer and hailin0 and removed request for hailin0 and TyrantLucifer December 13, 2022 11:06
@whutpencil
Copy link
Contributor

@TaoZex hi, I have some question, can you add me wechat: spider360, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants