Skip to content

Auto Instrumentation not Supporting Flink #13686

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

Open
dfercode opened this issue Apr 10, 2025 · 1 comment
Open

Auto Instrumentation not Supporting Flink #13686

dfercode opened this issue Apr 10, 2025 · 1 comment
Labels
contribution welcome Request makes sense, maintainers probably won't have time, contribution would be welcome enhancement New feature or request new instrumentation

Comments

@dfercode
Copy link

Is your feature request related to a problem? Please describe.

Use java annotation @WithSpan and @SpanAttribute in flink application operators(e.g. keyby operator) will create new trace rather than use same trace, seems ot java agent is not able to injectt trace_id span_id into flink data piplines.
Suggestions:
e.g. flink operator will be source from kafka -> filter(@WithSpan) -> keyby ->map(@WithSpan) ->sink
same massage will generate new traces in fliter operator and map operator when message pass throught flink pippline, but I want same msg generated spans just have one trace_id

Describe the solution you'd like

  1. support auto instrumentation in flink
  2. support set remote trace_id and span_id in @SpanAttribute, so that new span can link to remote trace.

Describe alternatives you've considered

No response

Additional context

No response

@dfercode dfercode added enhancement New feature or request needs triage New issue that requires triage labels Apr 10, 2025
@laurit
Copy link
Contributor

laurit commented Apr 10, 2025

We'd welcome a contribution for this. You should consider whether support for distributed tracing could be better implemented inside flink.

@laurit laurit added contribution welcome Request makes sense, maintainers probably won't have time, contribution would be welcome new instrumentation and removed needs triage New issue that requires triage labels Apr 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contribution welcome Request makes sense, maintainers probably won't have time, contribution would be welcome enhancement New feature or request new instrumentation
Projects
None yet
Development

No branches or pull requests

2 participants