fix(resource-manager): do not register processor if tracing disabled

[hassiebp]

Important

Add tracing_enabled parameter to conditionally initialize tracing in LangfuseResourceManager and Langfuse client.

• Behavior:
  • Add tracing_enabled parameter to LangfuseResourceManager and Langfuse constructor.
  • Modify _initialize_instance() in resource_manager.py to conditionally initialize tracer and span processor based on tracing_enabled.
  • In client.py, pass tracing_enabled to LangfuseResourceManager and use NoOpTracer if tracing is disabled.
• Misc:
  • Update LangfuseResourceManager.__new__() to default tracing_enabled to True if not specified.

^{This description was created by for ae7ca32. You can customize this summary. It will automatically update as commits are pushed.}

Greptile Summary

Disclaimer: Experimental PR review

Optimizes the Langfuse client by conditionally initializing OpenTelemetry tracing based on configuration, preventing unnecessary processor registration when tracing is disabled.

• Added tracing_enabled parameter to LangfuseResourceManager in langfuse/_client/resource_manager.py for conditional tracing initialization
• Modified Langfuse client in langfuse/_client/client.py to properly propagate tracing configuration
• Implementation maintains thread-safety and backward compatibility while reducing overhead when tracing is disabled

[greptile-apps]

LGTM

_{2 files reviewed, no comments
Edit PR Review Bot Settings | Greptile}