Skip to content

Comments

feat: API pipeline run list has execution summary#96

Open
yuechao-qin wants to merge 1 commit intoycq/function_get_execution_status_statsfrom
ycq/api_pipeline_run_list_has_ended
Open

feat: API pipeline run list has execution summary#96
yuechao-qin wants to merge 1 commit intoycq/function_get_execution_status_statsfrom
ycq/api_pipeline_run_list_has_ended

Conversation

@yuechao-qin
Copy link
Collaborator

@yuechao-qin yuechao-qin commented Feb 17, 2026

TL;DR

Added execution summary to pipeline run responses to provide aggregated information about pipeline execution status (GET /api/pipeline_runs/)..

What changed?

  • Added execution_summary field to PipelineRunResponse dataclass
  • Renamed _get_execution_status_stats to _get_execution_stats_and_summary to return both stats and summary

How to test?

  1. Create a pipeline run with multiple nodes in different states
  2. Call the list API with include_execution_stats=True
  3. Verify the response contains both execution stats and execution summary
  4. Check that the summary correctly reports total nodes, ended nodes, and completion status

Run the new test cases in tests/test_api_server_sql.py:

pytest tests/test_api_server_sql.py

Why make this change?

This closes #87

image.png

Copy link
Collaborator Author

yuechao-qin commented Feb 17, 2026

@yuechao-qin yuechao-qin marked this pull request as ready for review February 17, 2026 17:40
@yuechao-qin yuechao-qin requested a review from Ark-kun as a code owner February 17, 2026 17:40
@yuechao-qin yuechao-qin changed the base branch from ycq/function_get_execution_status_stats to graphite-base/96 February 18, 2026 02:26
@yuechao-qin yuechao-qin force-pushed the ycq/api_pipeline_run_list_has_ended branch from ea578e3 to 489c797 Compare February 18, 2026 02:26
@yuechao-qin yuechao-qin changed the base branch from graphite-base/96 to ycq/function_get_execution_status_stats February 18, 2026 02:27
@yuechao-qin yuechao-qin changed the base branch from ycq/function_get_execution_status_stats to graphite-base/96 February 20, 2026 04:27
@yuechao-qin yuechao-qin force-pushed the ycq/api_pipeline_run_list_has_ended branch from 489c797 to 3df5bde Compare February 20, 2026 04:28
@yuechao-qin yuechao-qin changed the base branch from graphite-base/96 to ycq/function_get_execution_status_stats February 20, 2026 04:28
@yuechao-qin yuechao-qin force-pushed the ycq/function_get_execution_status_stats branch 2 times, most recently from e382bc8 to 5ae141c Compare February 21, 2026 02:41
@yuechao-qin yuechao-qin force-pushed the ycq/api_pipeline_run_list_has_ended branch from 3df5bde to 5014f69 Compare February 21, 2026 02:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant