Skip to content

fix: lazy fill workflow task release plans#4734

Open
huanghongbo-hhb wants to merge 1 commit into
koderover:mainfrom
huanghongbo-hhb:feat/workflow-task-release-plan-lazyfill-main
Open

fix: lazy fill workflow task release plans#4734
huanghongbo-hhb wants to merge 1 commit into
koderover:mainfrom
huanghongbo-hhb:feat/workflow-task-release-plan-lazyfill-main

Conversation

@huanghongbo-hhb

@huanghongbo-hhb huanghongbo-hhb commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

Summary

  • lazy fill release plan metadata for historical workflow task records in workflow history APIs

Why

  • after the release plan column was enabled, historical workflow tasks still could not show their related release plans until a new release plan execution happened
  • the existing direct workflow_task.release_plan field only covers tasks created after the feature landed

Main Changes

  • add a release plan repository query that maps the current workflow and current page task IDs to release plan refs
  • lazy fill missing release_plan data in workflow task list and detail APIs instead of waiting for a new execution
  • asynchronously persist the resolved release plan ref back to workflow_task so later requests can hit the direct field

Risk / Compatibility

  • low risk: this only affects workflow history release plan enrichment for tasks missing release_plan
  • new tasks still use the existing direct write path; old tasks are enriched from existing release plan task associations

Test

  • tests not run

Contact


This change is Reviewable

@huanghongbo-hhb huanghongbo-hhb force-pushed the feat/workflow-task-release-plan-lazyfill-main branch from da58cf1 to 5475f39 Compare June 2, 2026 06:17
Signed-off-by: huanghongbo-hhb <huanghongbo@koderover.com>
@huanghongbo-hhb huanghongbo-hhb force-pushed the feat/workflow-task-release-plan-lazyfill-main branch from 5475f39 to 3d48bbf Compare June 2, 2026 06:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant