NotificationsYou must be signed in to change notification settings
Fork746
Star10.7k

Migrate getWorkflowEvaluationRunEpisodesByTaskName and countWorkflowE…#5287

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Draft

shuyangli wants to merge1 commit intomain

base:main

Choose a base branch

fromsl/migrate-workflow-eval-apis

Draft

Migrate getWorkflowEvaluationRunEpisodesByTaskName and countWorkflowE…#5287

shuyangli wants to merge1 commit intomainfromsl/migrate-workflow-eval-apis

+1,439 −251

Conversation

Copy link

Member

shuyangli commentedDec 18, 2025

…valuationRunEpisodesByTaskName to backend

This moves the workflow evaluation episodes by task name queries from the UI (direct ClickHouse calls) to the backend (Rust API endpoints).

Backend changes:

Add GroupedWorkflowEvaluationRunEpisodeWithFeedbackRow type to workflow_evaluation_queries.rs
Add list_workflow_evaluation_run_episodes_by_task_name and count_workflow_evaluation_run_episode_groups_by_task_name trait methods
Implement ClickHouse queries with complex CTEs for grouping by task_name
Add unit tests using MockWorkflowEvaluationQueries
Add e2e database tests in tests/e2e/db/workflow_evaluation_queries.rs
Add route handlers at /internal/workflow-evaluations/episodes-by-task-name and /internal/workflow-evaluations/episodes-by-task-name/count
Add e2e endpoint tests (require gateway restart)

Frontend changes:

Generate and export TypeScript bindings for new types
Add listWorkflowEvaluationRunEpisodesByTaskName and countWorkflowEvaluationRunEpisodeGroupsByTaskName methods to TensorZeroClient
Update route.tsx to use new client methods instead of direct ClickHouse calls
Delete unused getWorkflowEvaluationRunEpisodesByTaskName and countWorkflowEvaluationRunEpisodesByTaskName functions from workflow_evaluations.server.ts
Delete GroupedWorkflowEvaluationRunEpisodeWithFeedback Zod schema and type
Remove corresponding test for deleted function

Migrate getWorkflowEvaluationRunEpisodesByTaskName and countWorkflowE…

6c30994

…valuationRunEpisodesByTaskName to backendThis moves the workflow evaluation episodes by task name queries from theUI (direct ClickHouse calls) to the backend (Rust API endpoints).Backend changes:- Add GroupedWorkflowEvaluationRunEpisodeWithFeedbackRow type to workflow_evaluation_queries.rs- Add list_workflow_evaluation_run_episodes_by_task_name and  count_workflow_evaluation_run_episode_groups_by_task_name trait methods- Implement ClickHouse queries with complex CTEs for grouping by task_name- Add unit tests using MockWorkflowEvaluationQueries- Add e2e database tests in tests/e2e/db/workflow_evaluation_queries.rs- Add route handlers at /internal/workflow-evaluations/episodes-by-task-name  and /internal/workflow-evaluations/episodes-by-task-name/count- Add e2e endpoint tests (require gateway restart)Frontend changes:- Generate and export TypeScript bindings for new types- Add listWorkflowEvaluationRunEpisodesByTaskName and  countWorkflowEvaluationRunEpisodeGroupsByTaskName methods to TensorZeroClient- Update route.tsx to use new client methods instead of direct ClickHouse calls- Delete unused getWorkflowEvaluationRunEpisodesByTaskName and  countWorkflowEvaluationRunEpisodesByTaskName functions from workflow_evaluations.server.ts- Delete GroupedWorkflowEvaluationRunEpisodeWithFeedback Zod schema and type- Remove corresponding test for deleted function