Code review comment for ~cgrabowski/maas:fix_connections_dict_update_while_updating_connection_count

Revision history for this message
Christian Grabowski (cgrabowski) wrote :

Hmm I suppose that is possible, we could use a lock instead, but in order to address that particular issue, I believe we'd have to lock for the entire registerRPCWorkerConnection() and _updateConnection(), which might be fine, but it might be a problem if many connections come in at once.

« Back to merge proposal