Start using transaction decorator instead of `run_in_transaction`

Replace all usages of run_in_transaction with the new run_in_transaction_wrapper.

List of files

• email_manager.py @Nik-09
• feedback_jobs_continuous.py @Nik-09
• feedback_services.py @lkbhitesh07
• question_services.py @lkbhitesh07
• rating_services.py @vojtechjelinek (#11741)
• stats_services.py @lkbhitesh07
• suggestion_jobs_one_off.py @lkbhitesh07
• suggestion_services.py @lkbhitesh07
• user_services.py @lkbhitesh07
• wipeout_services.py @OBITORASU
• jobs.py @lkbhitesh07
• base_model/gae_models.py @OBITORASU
• statistics/gae_models.py @lkbhitesh07
• user_jobs_continuous.py @lkbhitesh07

Steps to refactor

1. Claim one of the files listed above by pinging @vojtechjelinek.
2. Look at the occurrences of run_in_transaction in the claimed file.
3. Instead of doing for example transaction_services.run_in_transaction(_update_user_rating) on top of the definition of _update_user_rating the @transaction_services.run_in_transaction_wrapper decorator should be added. In the callsite, the transaction_services.run_in_transaction(_update_user_rating) should be replaced with _update_user_rating_transactional().
4. See #11741 for reference

After all occurences are replaced remame the run_in_transaction_wrapper to run_in_transaction.

[vojtechjelinek]

@OBITORASU The BaseModel.delete_multi can be used in other files, also since some models inherit from BaseModel (like UserSettingsModel), the delete_multi can be used in these classes too.

[OBITORASU]

Also another question regarding the wipeout_services.py Just to confirm, is it wipeout_services.py or wipeout_service.py ? Apparently I can’t fuzzy find wipeout_services.py the closest file to it is wipeout_service.py which I can find.