Remove prefix completely

commanded / commanded-extreme-adapter

Extreme event store adapter for Commanded

MIT License

12 stars 18 forks source link

Remove prefix completely #17

Open drozzy opened 6 years ago

drozzy commented 6 years ago

I got rid of the prefix, replacing it with the $streams stream. This works for loading aggregates pretty well, which is all I really need. Allows me to do projections in Greg's EventStore correctly (e.g. by using $ce-aggregatename):

Did not test:

event handlers
process managers

drozzy commented 6 years ago

This certainly breaks tests.. not sure how to fix them. E.g.:

5) test subscribe to all streams should receive events appended to any stream (Commanded.EventStore.Adapter.SubscriptionTest)
     deps/commanded/test/event_store_adapter/subscription_test.exs:105
     No message matching {:events, received_events} after 5000ms.
     The process mailbox is empty.

drozzy commented 6 years ago

I also added a fix that set the extreme dep to ~> 0.11.0