Skip to content

Add option for Transform on Request to AppNote0022#182

Open
j616 wants to merge 5 commits into
jamessa-transcodeArchfrom
jamessa-virtualTanscode
Open

Add option for Transform on Request to AppNote0022#182
j616 wants to merge 5 commits into
jamessa-transcodeArchfrom
jamessa-virtualTanscode

Conversation

@j616
Copy link
Copy Markdown
Contributor

@j616 j616 commented Apr 15, 2026

Details

This PR adds an additional transform architecture to AppNote0022, for transcode on request. Or Just In Time transform. As this architecture is more experimental, it has been split into its own PR.

Jira Issue (if relevant)

Jira URL: https://jira.dev.bbc.co.uk/browse/CLOUDFIT-5511

Related PRs

Depends on #181

Submitter PR Checks

(tick as appropriate)

  • PR completes task/fixes bug
  • API version has been incremented if necessary
  • ADR status has been updated, and ADR implementation has been recorded
  • Documentation updated (README, etc.)
  • PR added to Jira Issue (if relevant)
  • Follow-up stories added to Jira

Reviewer PR Checks

(tick as appropriate)

  • PR completes task/fixes bug
  • Design makes sense, and fits with our current code base
  • Code is easy to follow
  • PR size is sensible
  • Commit history is sensible and tidy

Info on PRs

The checks above are guidelines. They don't all have to be ticked, but they should all have been considered.

@j616 j616 requested a review from a team as a code owner April 15, 2026 15:51
Copy link
Copy Markdown
Member

@samdbmg samdbmg left a comment

Choose a reason for hiding this comment

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

Looks good - I've suggested a couple of small wording tweaks

Comment thread docs/appnotes/0022-transform-architectures.md
Comment thread docs/appnotes/0022-transform-architectures.md
Comment thread docs/appnotes/0022-transform-architectures.md
Comment thread docs/appnotes/0022-transcode-architectures.md Outdated
@samdbmg samdbmg linked an issue Apr 29, 2026 that may be closed by this pull request
@j616 j616 force-pushed the jamessa-virtualTanscode branch from 31cffda to c46d2ef Compare May 13, 2026 15:42
@j616 j616 force-pushed the jamessa-transcodeArch branch from 18f3216 to bc7364f Compare May 13, 2026 15:42
@j616 j616 force-pushed the jamessa-virtualTanscode branch 3 times, most recently from 57b2bc1 to 09b5a59 Compare May 15, 2026 16:15
@j616 j616 changed the title Add option for Transcode on Request to AppNote0022 Add option for Transform on Request to AppNote0022 May 15, 2026
@j616 j616 force-pushed the jamessa-transcodeArch branch from 41d8a5e to 67b30ac Compare May 15, 2026 16:20
@j616 j616 force-pushed the jamessa-virtualTanscode branch from 09b5a59 to d491f30 Compare May 15, 2026 16:23
@j616 j616 requested a review from samdbmg May 15, 2026 16:25
Copy link
Copy Markdown
Member

@samdbmg samdbmg left a comment

Choose a reason for hiding this comment

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

Changes LGTM

Comment thread docs/appnotes/0022-transform-architectures.md
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.

Describe architectures for implementing transcode

2 participants