Skip to content

feat(dataset-run-items): support reads for public APIs #8179

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 13 commits into from
Jul 30, 2025

Conversation

marliessophie
Copy link
Member

@marliessophie marliessophie commented Jul 29, 2025

Important

Add support for reading dataset run items via public APIs using Clickhouse, enhancing data retrieval with filtering, ordering, and pagination.

  • Public API Enhancements:
    • Added generateDatasetRunItemsForPublicApi and getDatasetRunItemsCountForPublicApi in dataset-run-items.ts to fetch dataset run items and their count using Clickhouse.
    • Updated GET /dataset-run-items and GET /datasets/{name}/runs/{runName} endpoints to use new functions for Clickhouse execution.
  • Repository Changes:
    • Added getDatasetRunItemsByDatasetIdCh and getDatasetRunItemsCountByDatasetIdCh in dataset-run-items.ts for Clickhouse queries.
    • Implemented getDatasetRunItemsTableInternal to handle internal query logic with filtering and ordering.
  • Schema and Table Definitions:
    • Added dataset_run_items to ClickhouseTableNames in schema.ts.
    • Created mapDatasetRunItemsTable.ts for UI column mappings.
  • Transformations:
    • Added transformDbDatasetRunItemToAPIDatasetRunItemCh in datasets.ts for transforming Clickhouse data to API format.

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

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. feat-datasets labels Jul 29, 2025
@langfuse langfuse deleted a comment from ellipsis-dev bot Jul 29, 2025
@langfuse langfuse deleted a comment from ellipsis-dev bot Jul 29, 2025
@langfuse langfuse deleted a comment from ellipsis-dev bot Jul 29, 2025
@marliessophie marliessophie changed the title chore(dataset-run-items): GET /dataset-run-items feat(dataset-run-items): support reads for public APIs Jul 29, 2025
…atest version of each dataset run item is retrieved
@langfuse langfuse deleted a comment from ellipsis-dev bot Jul 29, 2025
@langfuse langfuse deleted a comment from ellipsis-dev bot Jul 29, 2025
@marliessophie marliessophie added this pull request to the merge queue Jul 30, 2025
@dosubot dosubot bot added the auto-merge This PR is set to be merged label Jul 30, 2025
Merged via the queue into main with commit df57ff6 Jul 30, 2025
28 checks passed
@marliessophie marliessophie deleted the marlies/lfe-6023 branch July 30, 2025 14:07
@dosubot dosubot bot removed the auto-merge This PR is set to be merged label Jul 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat-datasets size:L This PR changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants