reconsider lock type

[smcquay]

I don't believe that pointers are what we want here:

https://github.com/ContextLogic/eventmaster/blob/331be5b624ea02fb5e0ebea21fc30385b3610f6e/event_store.go#L86

since the method receivers on EventStore are already pointers the extra indirection is superfluous.