Skip to content

chore: upsert traces into both traces tables #8268

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

Merged
merged 3 commits into from
Aug 4, 2025

Conversation

Steffen911
Copy link
Contributor

@Steffen911 Steffen911 commented Aug 1, 2025

Important

Adds conditional insertion of traces into traces_mt table in upsertTrace based on new environment variable LANGFUSE_EXPERIMENT_INSERT_INTO_AGGREGATING_MERGE_TREES.

  • Environment:
    • Adds LANGFUSE_EXPERIMENT_INSERT_INTO_AGGREGATING_MERGE_TREES to env.ts with default false.
  • Functionality:
    • Updates upsertTrace in traces.ts to insert into traces_mt if LANGFUSE_EXPERIMENT_INSERT_INTO_AGGREGATING_MERGE_TREES is true.
    • Converts trace data to traces_mt format using convertTraceToTraceMt before insertion.
    • Uses clickhouseClient for direct insertion into traces_mt.

This description was created by Ellipsis for f78e564. You can customize this summary. It will automatically update as commits are pushed.

@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Aug 1, 2025
@Steffen911 Steffen911 merged commit 075836f into main Aug 4, 2025
28 checks passed
@Steffen911 Steffen911 deleted the steffen/lfe-5580-handle-trace-upsert branch August 4, 2025 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:M This PR changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant