Makes InMemoryEventLogStorage a ConfigurableClass so it can be used via overrides in DagsterInstance.local_temp to have a test instance that does not persist events to sqlite.
Seems fine. My only request is better naming to indicate why both EphemeralEventLogStorage and InMemoryEventLogStorage exist.
This is also why ConfigurableClass should be a distinct layer that is more functional in nature. Quite the contortion here for what should just a function that creates something.
to be clear the only purpose that this class exists is to deal with the ConfigurableClass stuff?