hail icon indicating copy to clipboard operation
hail copied to clipboard

[batch] Add updates to batches

Open daniel-goldstein opened this issue 1 year ago • 3 comments

An attempt to break open batches into a bunch of smaller PRs

daniel-goldstein avatar Sep 01 '22 23:09 daniel-goldstein

Can I also see the changes that follow after this set of PRs with the interaction between the client and how you account for update ids that aren't hardcoded to be 1?

Yes I'll put up a PR and tag you in it.

daniel-goldstein avatar Sep 13 '22 17:09 daniel-goldstein

I just noticed that we didn't add a foreign key constraint on the jobs table for the batch update. I think unfortunately we should add it... thoughts?

jigold avatar Sep 15 '22 20:09 jigold

Ah yes, that was my mistake forgetting to add it in. I think these should all have FK constraints

daniel-goldstein avatar Sep 15 '22 21:09 daniel-goldstein

Got broken up and sent to relevant PRs

daniel-goldstein avatar Sep 27 '22 20:09 daniel-goldstein