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

[Bug][Sort] redundant&conflicts dependencys of OpenTelemetryLogger #11422

Closed
2 tasks done
qy-liuhuo opened this issue Oct 28, 2024 · 0 comments · Fixed by #11423
Closed
2 tasks done

[Bug][Sort] redundant&conflicts dependencys of OpenTelemetryLogger #11422

qy-liuhuo opened this issue Oct 28, 2024 · 0 comments · Fixed by #11423
Assignees
@qy-liuhuo
Labels
type/bug Something is wrong
Milestone
2.1.0

Comments

@qy-liuhuo
Copy link
Contributor

qy-liuhuo commented Oct 28, 2024 •
edited
Loading

What happened

OpenTelemetryLogger adds some opentelemetry-related maven dependency packages to the sort-connector-base module.

However, these packages have some redundancy and conflicts with inlong's original opentelemetry packages.

This results in unit tests failing when integrating some connectors（https://github.com/apache/inlong/pull/11351#issuecomment-2425694927）

What you expected to happen

Remove redundant and conflicting dependencies

How to reproduce

Run sort-end-to-end-tests-v1.15

Environment

No response

InLong version

master

InLong Component

InLong Sort

Are you willing to submit PR?

  • Yes, I am willing to submit a PR!

Code of Conduct
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@qy-liuhuo qy-liuhuo

Labels
type/bug Something is wrong
Projects
None yet
Milestone
2.1.0
Development

Successfully merging a pull request may close this issue.

[INLONG-11422][Sort] Redundant&conflicts dependencys of OpenTelemetryLogger qy-liuhuo/inlong
2 participants
@dockerzhang @qy-liuhuo