Skip to content

[Improve][Connector-V2] oracle cdc e2e optimization #6232

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
Merged
Hisoka-X merged 8 commits into from
Feb 19, 2024
Merged

[Improve][Connector-V2] oracle cdc e2e optimization #6232

Hisoka-X merged 8 commits into from
Feb 19, 2024

Conversation

@Carl-Zhou-CN
Copy link
Member

@Carl-Zhou-CN Carl-Zhou-CN commented Jan 17, 2024

Purpose of this pull request

oracle cdc e2e optimization

Does this PR introduce any user-facing change?

How was this patch tested?

Check list

@Hisoka-X
Copy link
Member

Hisoka-X commented Jan 18, 2024 •
edited
Loading

I believe the reason is connection leak. Not resource not enough.
image

@Carl-Zhou-CN
Copy link
Member Author

Carl-Zhou-CN commented Jan 18, 2024

@Hisoka-X Yeah, it shouldn't be about resources

zhilinli123
zhilinli123 reviewed Jan 19, 2024
View reviewed changes
.github/workflows/backend.yml Outdated
java-version: ${{ matrix.java }}
distribution: 'temurin'
cache: 'maven'
- name: run doris connector integration test
Copy link
Contributor

@zhilinli123 zhilinli123 Jan 19, 2024

Choose a reason for hiding this comment

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

doris?

Copy link
Member Author

@Carl-Zhou-CN Carl-Zhou-CN Jan 19, 2024

Choose a reason for hiding this comment

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

done

hailin0
hailin0 reviewed Feb 18, 2024
View reviewed changes
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

cc @Hisoka-X @EricJoy2048

@Hisoka-X Hisoka-X merged commit 92f847c into apache:dev Feb 19, 2024
chaorongzhi pushed a commit to chaorongzhi/seatunnel that referenced this pull request Aug 21, 2024
@Carl-Zhou-CN
[Improve][Connector-V2] oracle cdc e2e optimization (apache#6232)
5cbedaa
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hailin0 hailin0 hailin0 approved these changes

@Hisoka-X Hisoka-X Hisoka-X approved these changes

+1 more reviewer

@zhilinli123 zhilinli123 zhilinli123 left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

approved e2e oracle reviewed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Carl-Zhou-CN @Hisoka-X @hailin0 @zhilinli123