Skip to content

chore: add env to move short-term reads to AMTs #8367

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 2 commits into from
Aug 7, 2025

Conversation

Steffen911
Copy link
Contributor

@Steffen911 Steffen911 commented Aug 7, 2025

Important

Add LANGFUSE_EXPERIMENT_RETURN_NEW_RESULT_SHORT_TERM for short-term read handling using AMTs and update measureAndReturn and related functions to support it.

  • Environment:
    • Add LANGFUSE_EXPERIMENT_RETURN_NEW_RESULT_SHORT_TERM to env.ts for short-term read handling.
  • Functionality:
    • Update measureAndReturn in measureAndReturn.ts to handle short-term reads using AMTs if LANGFUSE_EXPERIMENT_RETURN_NEW_RESULT_SHORT_TERM is "true" and minStartTime is within 30 days.
    • Modify getObservationsTableInternal, checkTraceExists, getTracesBySessionId, getTraceCountsByProjectInCreationInterval, getTraceCountOfProjectsSinceCreationDate, getTracesGroupedByName, getTracesGroupedByUsers, getTracesGroupedByTags, getTotalUserCount, getUserMetrics, getSessionsTableGeneric, getTracesTableGeneric, generateTracesForPublicApi, and executeQuery to pass minStartTime to measureAndReturn.
  • Misc:
    • Remove deletion from traces_null in deleteTracesOlderThanDays in observations.ts.

This description was created by Ellipsis for d662ed9. 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 7, 2025
@Steffen911 Steffen911 enabled auto-merge August 7, 2025 15:12
@dosubot dosubot bot added the auto-merge This PR is set to be merged label Aug 7, 2025
@Steffen911 Steffen911 added this pull request to the merge queue Aug 7, 2025
Merged via the queue into main with commit 10fdd9e Aug 7, 2025
28 checks passed
@Steffen911 Steffen911 deleted the steffen/allow-short-term-amt-reads branch August 7, 2025 15:48
@dosubot dosubot bot removed the auto-merge This PR is set to be merged label Aug 7, 2025
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