AggregateManager::handle command is generic in Error

[cottinisimone]

• I changed some stuff in order to let this work in lira. There's only few small changes. I wasn't able to find a way to let this work with async_graphql using the store as Box<dyn EventStore<Aggregate = TheAggregate, Error = PgStoreError> + Send + Sync> since it ended up with this error:

error: higher-ranked lifetime error
  --> src/web/graphql/internal/mutation.rs:23:1
   |
23 | #[Object]
   | ^^^^^^^^^
   |
   = note: could not prove `Pin<Box<[async block@src/web/graphql/internal/mutation.rs:23:1: 23:10]>>: CoerceUnsized<Pin<Box<(dyn futures::Future<Output = std::result::Result<std::option::Option<async_graphql::Value>, ServerError>> + std::marker::Send + 'i)>>>`
   = note: this error originates in the attribute macro `Object` (in Nightly builds, run with -Z macro-backtrace for more info)

error: higher-ranked lifetime error
  --> src/web/graphql/public/mutation.rs:17:1
   |
17 | #[Object]
   | ^^^^^^^^^
   |
   = note: could not prove `Pin<Box<[async block@src/web/graphql/public/mutation.rs:17:1: 17:10]>>: CoerceUnsized<Pin<Box<(dyn futures::Future<Output = std::result::Result<std::option::Option<async_graphql::Value>, ServerError>> + std::marker::Send + 'i)>>>`
   = note: this error originates in the attribute macro `Object` (in Nightly builds, run with -Z macro-backtrace for more info)

It seems to be related in some way to how async_trait handles lifetimes. It will follow further investigations since the possibility to use a generic EventStore seems to me to be important

[angelo-rendina-prima]

I believe I understand your issue was that async_graphql is complaining about Box<dyn Error>, but that same type is still around so I am not sure how this change solves your problem :(