Update completed endpoint with new job group format - Githubissues

populationgenomics / hail

Scalable genomic data analysis.

https://hail.is

MIT License

1 stars 1 forks source link

Update completed endpoint with new job group format #328

Closed illusional closed 6 months ago

illusional commented 6 months ago

The latest change #324 (upstream from head) introduced a stronger job group concept which included a fairly sizeable migration, this new structure broke our /completed endpoint which we use for Hail / Seqr aggregation.

Rewrite the function to align back with the original function it was modelled off: https://github.com/populationgenomics/hail/blob/01714cc51f092304eb7b4de47bdc2c7c192b13cd/batch/batch/front_end/front_end.py#L1714-L1739

I haven't tested the endpoint, but I've tested the query and it works.

Columns selected:

id
userdata
user
billing_project
attributes
callback
deleted
n_jobs
time_created
time_completed
msec_mcpu
token
state
format_version
time_closed
cancel_after_n_failures
migrated_batch
cancelled
n_completed
n_succeeded
n_failed
n_cancelled
cost
cost_breakdown

illusional commented 6 months ago

Yeah definitely haha, I want to keep it as close to the referenced code in this case so I can find it next time I break it :)