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] Only one OpenTelemetryAppender should be registered #11455

Closed
2 tasks done
qy-liuhuo opened this issue Nov 5, 2024 · 0 comments · Fixed by #11480
Closed
2 tasks done

[Bug][Sort] Only one OpenTelemetryAppender should be registered #11455

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

Comments

@qy-liuhuo
Copy link
Contributor

What happened

The current OpenTelemetryLogger may generate multiple OpenTelemetryAppenders, which may cause duplicate log reporting.

What you expected to happen

Only one OpenTelemetryAppender should be registered

How to reproduce

Some connectors need to instantiate multiple sourceReaders, which will cause multiple identical OpenTelemetryAppenders to be registered in the LoggerContext.

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-11455][Sort] Only one OpenTelemetryAppender should be registered qy-liuhuo/inlong
2 participants
@dockerzhang @qy-liuhuo