tree a4fac9d07c3ef03adc4d79358dcff05aba9cbf59
parent ce4e7c6609a6aaa51745350e2a6b0661793861b3
author Alice Kober-Sotzek <aliceks@google.com> 1600421115 +0000
committer Alice Kober-Sotzek <aliceks@google.com> 1600421115 +0000

Revert "Migrate all emails to send async via asyncPostUpdate"

This reverts commit ce4e7c6609a6aaa51745350e2a6b0661793861b3.

Reason for revert:
The code has threading issues. For instance, we see an NPE for submitter in SubmitStrategyOp#asyncPostUpdate on real servers. Looking at the code, there doesn't seem to be any precautions for the use of the existing objects on different threads. Objects like SubmitStrategyOp are not built to be thread-safe at the moment. We can't simply pass them to another thread and expect them to work correctly.

Change-Id: If2e8c7c4d40231fdc9390f52c883401a1c0710cb