Retry celery patron sync on StaleDataError (PP-2005)

[jonathangreen]

Description

Retry when we get a StaleDataError while running a patron sync.

Motivation and Context

Looking at the cloudwatch alarms we are getting for failed Celery tasks, it looks like almost all of them are:

sqlalchemy.orm.exc.StaleDataError: UPDATE statement on table 'loans' expected to update 1 row(s); 0 were matched.

These are probably mostly unavoidable, due to something happening (like a checkout or checkin) while the sync task is running. Just retrying should solve the issue.

How Has This Been Tested?

• Added unit tests

Checklist

• I have updated the documentation accordingly.
• All new and existing tests passed.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.09%. Comparing base (0ff6c7d) to head (435e512).
Report is 4 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2207   +/-   ##
=======================================
  Coverage   91.09%   91.09%           
=======================================
  Files         363      363           
  Lines       41224    41227    +3     
  Branches     8834     8835    +1     
=======================================
+ Hits        37553    37556    +3     
  Misses       2406     2406           
  Partials     1265     1265           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[dbernstein]

🚀