Skip to content

feat: API Server - Pipeline runs - Made it possible to get pipeline run execution summary when using the get method#97

Merged
yuechao-qin merged 1 commit intomasterfrom
ycq/api_pipeline_run_get_has_ended
Apr 27, 2026
Merged

feat: API Server - Pipeline runs - Made it possible to get pipeline run execution summary when using the get method#97
yuechao-qin merged 1 commit intomasterfrom
ycq/api_pipeline_run_get_has_ended

Conversation

@yuechao-qin
Copy link
Copy Markdown
Collaborator

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

TL;DR

Added execution statistics to the pipeline run get endpoint (GET /api/pipeline_runs/{id}).

What changed?

  • Enhanced the get method to optionally include execution statistics
  • Added a new parameter include_execution_stats (default: False) to the get method
  • Extracted the execution stats population logic into a new helper method _populate_execution_stats

How to test?

  1. Call the pipeline run get endpoint with include_execution_stats=True
  2. Verify that the response includes execution status statistics and summary
  3. Run the new tests in TestPipelineRunServiceGet class to ensure the functionality works as expected

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

pytest tests/test_api_server_sql.py

Why make this change?

image.png

Comment thread cloud_pipelines_backend/api_server_sql.py
@yuechao-qin yuechao-qin force-pushed the ycq/api_pipeline_run_get_has_ended branch from 96dfb6c to 034f57b Compare February 20, 2026 04:28
@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 requested a review from Ark-kun February 20, 2026 04:33
@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
@yuechao-qin yuechao-qin force-pushed the ycq/api_pipeline_run_get_has_ended branch from 034f57b to b25a858 Compare February 21, 2026 02:41
Comment thread cloud_pipelines_backend/api_server_sql.py
Comment thread cloud_pipelines_backend/api_server_sql.py
Comment thread cloud_pipelines_backend/api_server_sql.py
@yuechao-qin yuechao-qin changed the base branch from ycq/api_pipeline_run_list_has_ended to graphite-base/97 April 4, 2026 04:50
@yuechao-qin yuechao-qin requested a review from morgan-wowk April 4, 2026 05:03
@yuechao-qin yuechao-qin force-pushed the ycq/api_pipeline_run_get_has_ended branch from b25a858 to 8bb91aa Compare April 4, 2026 05:17
@yuechao-qin yuechao-qin changed the base branch from graphite-base/97 to ycq/api_pipeline_run_list_has_ended April 4, 2026 05:17
Comment thread cloud_pipelines_backend/api_server_sql.py
Copy link
Copy Markdown
Contributor

@Ark-kun Ark-kun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved, but please resolve the threads.

@Ark-kun Ark-kun changed the title feat: API pipeline run get has execution summary feat: API Server - Pipeline runs - Made it possible to get pipeline run execution summary when using the get method Apr 17, 2026
@yuechao-qin yuechao-qin force-pushed the ycq/api_pipeline_run_list_has_ended branch from cec5214 to 4edaac6 Compare April 22, 2026 14:14
@yuechao-qin yuechao-qin force-pushed the ycq/api_pipeline_run_get_has_ended branch from 8bb91aa to 3524696 Compare April 22, 2026 14:14
Copy link
Copy Markdown
Collaborator Author

yuechao-qin commented Apr 27, 2026

Merge activity

  • Apr 27, 7:45 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Apr 27, 7:49 PM UTC: Graphite rebased this pull request as part of a merge.
  • Apr 27, 7:50 PM UTC: @yuechao-qin merged this pull request with Graphite.

@yuechao-qin yuechao-qin changed the base branch from ycq/api_pipeline_run_list_has_ended to graphite-base/97 April 27, 2026 19:46
@yuechao-qin yuechao-qin changed the base branch from graphite-base/97 to master April 27, 2026 19:48
@yuechao-qin yuechao-qin force-pushed the ycq/api_pipeline_run_get_has_ended branch from 3524696 to a8d7b2d Compare April 27, 2026 19:49
@yuechao-qin yuechao-qin merged commit ccd3efa into master Apr 27, 2026
7 checks passed
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.

3 participants