HomeElementl

Remove pyrsistent

Description

Remove pyrsistent

Summary:
This was used in the in-memory event log for synchronization.
However this is only used and useful in the ephmeral instance, and I
don't think the in-memory event log needs to be thread-safe.
Theorectically someone could invoke execute pipeline in two different
threads sharing the same ephemeral instance, but I think that is quite
fringe.

If we feel strongly that the in-memory case should remain thread safe
(e.g. with an threaded executor) we can add some synchronizsation
primitives to this

Test Plan: BK

Reviewers: dgibson, alangenfeld

Reviewed By: dgibson

Differential Revision: https://dagster.phacility.com/D7452

Details

Provenance
schrocknAuthored on Apr 15 2021, 1:51 PM
Reviewer
dgibson
Differential Revision
D7452: Remove pyrsistent
Parents
R1:efe944ac9b4e: Move upload_logs to dagit
Branches
Unknown
Tags
Unknown