Ben Smith

Results 106 comments of Ben Smith

Unfortunately it's not currently supported. Can you give me an example of what kind of metadata you want to include? We could extend to process manager API to allow setting...

Pull request #389 provides access to enriched metadata in the [`Commanded.Event.Upcaster` protocol](https://hexdocs.pm/commanded/Commanded.Event.Upcaster.html). This could be used to copy any fields from the [`Commanded.EventStore.RecordedEvent`](https://hexdocs.pm/commanded/Commanded.EventStore.RecordedEvent.html) struct, such as the causation and correlation...

@Yamilquery Could you show so example code to demonstrate what you're trying to do?

Yes, it's worth making the table name configurable to allow the same database to be used for both events and read model projections. As a temporary workaround Ecto allows you...

@jdewar If an aggregate process cannot append events to its event stream then an error should be returned to the dispatch command function call. The guarantee is that an `:ok`...

The case where the aggregate's current version is less than the expected version is only really possible in a multi-node deployment where you have the same aggregate instance process running...

Yes, this is a good approach to unit testing process managers in isolation.

I think it was preferable to remove the dependency on `:elixir_uuid` and implement the default UUID generator using the v4 format by copying the UUID implementation.

> So should have I retained `Commanded.UUID` and remove `uniq_compat`? Yes, I think so as dropping the dependency removes the potential for version clashes.

@Freyskeyd I've done some exploratory coding with Broadway in the [broadway-show](https://github.com/slashdotdash/broadway-show) repo. The behaviour I was keen to test was being able to stop the pipeline producer on error. This...