sul-dlss / purl-fetcher

An HTTP API for querying and updating PURLs
0 stars 1 forks source link

Add object locking around updates + deletes #899

Closed cbeer closed 1 month ago

cbeer commented 1 month ago

Or maybe there's something else in the architecture that's stopping simultaneous updates?

justinlittman commented 1 month ago

Will second attempt to lock block or raise? (I don't love blocking, as it might lock up http requests. I'd prefer to see it raise so that the caller can retry.)

cbeer commented 1 month ago

We can make it do whatever. It's not yet clear to me whether this is needed (but also not clear that it isn't.). What do you think?

justinlittman commented 1 month ago

In theory it shouldn't happen. However, since on the DSA side publishing is performed by a job, anything is possible given retries. This seems lightweight enough that it is worth it.