willkg/kitsune — comment on commit 5565e77209872048d42a8a9157c628ff31539ff9
File: apps/search/es_utils.py line 67
By erikrose:

Really? It falls out of the for loop immediately, so it really just saves a clear. I still don't care for it. (Granted, I've always been an indentation-should-equal-control-flow person.) We should also pull _thread_local.es_index_task_set into a local so we don't have to do some the locking twice. And name _thread_local something more descriptive, like _local_tasks.