Changes between Version 1 and Version 2 of Ticket #37033, comment 2


Ignore:
Timestamp:
Apr 13, 2026, 7:55:56 PM (4 weeks ago)
Author:
Cameron Gorrie

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #37033, comment 2

    v1 v2  
    11Thanks for taking a look, Simon. Agreed that we're asking a lot of the libraries / frameworks here w.r.t. idempotency.
    22
    3 Warm shutdown seems like a good plan: check e.g. `self.terminate = True` periodically. In the context we're supporting this system, it would require us to change an uncomfortable amount of code, but it is doable. I thought about doing it in a thread, but I think `SystemExit` would have the exact same problem. I have a feeling we'd have options if we had async handlers, but we're not on 6.0 yet and even then it's a big change... can you think of anything I've missed?
     3Warm shutdown seems like a good plan: check e.g. `self.terminate = True` periodically. In the context we're supporting this system, it would require us to change an uncomfortable amount of code, but it is doable. I thought about doing it in a thread, but I think `SystemExit` would have the exact same problem. I have a feeling we'd have options if we had async handlers, but that would be a big change... can you think of anything I've missed?
    44
    55I've opened a ticket with psycopg and am about to with celery. At first look, I didn't see anything in the documentation that explicitly calls this out. I don't think this issue is resolved by writing code -- but to collaborate in ensuring the message is out to help those who--perhaps naively--believe this combination will work. I opened it here too because the root cause is two-fold:
Back to Top