Skip to content

feat: Add plain "status" field to the JobResponse#1497

Merged
milenkovicm merged 1 commit intoapache:mainfrom
martin-g:351-add-job-plain-status-field
Mar 10, 2026
Merged

feat: Add plain "status" field to the JobResponse#1497
milenkovicm merged 1 commit intoapache:mainfrom
martin-g:351-add-job-plain-status-field

Conversation

@martin-g
Copy link
Member

Which issue does this PR close?

Closes #351.

Rationale for this change

The old "job_status" is more like a status description with some extra information.
The plain "status" field just gives one of: Running, Queued, Completed or Failed

What changes are included in this PR?

The JobResponse entity now includes a "status" field which value is just the status name without any extra details. One of: "Running", "Queued", "Completed" or "Failed".

Are there any user-facing changes?

A new field is added to the JobResponse JSON in a backward compatible way

The old "job_status" is more like a status description with some extra
information.
The plain "status" field just gives one of: Running, Queued, Completed
or Failed
@milenkovicm milenkovicm self-requested a review March 10, 2026 14:49
Copy link
Contributor

@milenkovicm milenkovicm left a comment

Choose a reason for hiding this comment

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

this looks like a backward compatible change, you're adding new field (or im missing something), anyway its good change thanks

@milenkovicm
Copy link
Contributor

merge whenever ready @martin-g

@milenkovicm milenkovicm merged commit 2fccbfa into apache:main Mar 10, 2026
17 checks passed
@martin-g martin-g deleted the 351-add-job-plain-status-field branch March 11, 2026 06:05
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.

Expose job status enum in API

2 participants