fix(flushing): flush interval to millis in SpanProcessor

[hassiebp]

Important

Convert flush_interval to milliseconds in LangfuseSpanProcessor and update documentation for LANGFUSE_FLUSH_INTERVAL.

• Behavior:
  • Convert flush_interval from seconds to milliseconds in LangfuseSpanProcessor.__init__() in span_processor.py.
  • Update LANGFUSE_FLUSH_INTERVAL documentation in environment_variables.py to specify delay is in seconds.
• Misc:
  • Adjust schedule_delay_millis calculation in LangfuseSpanProcessor.__init__() to handle None values.

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

Greptile Summary

Disclaimer: Experimental PR review

Fixed critical timing issue in SpanProcessor by correcting unit conversion from seconds to milliseconds for flush intervals, ensuring proper batch processing timing.

• Fixed timing bug in langfuse/_client/span_processor.py where seconds were incorrectly interpreted as milliseconds in BatchSpanProcessor
• Updated langfuse/_client/environment_variables.py documentation to clarify LANGFUSE_FLUSH_INTERVAL is in seconds
• Added proper handling of None values for flush interval parameter

[greptile-apps]

_{2 files reviewed, 1 comment
Edit PR Review Bot Settings | Greptile}