-
-
Notifications
You must be signed in to change notification settings - Fork 130
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Question: Any intention to extend/fork this library to async? #233
Comments
Sorry for the delay in answering this. The short answer is yes. Beyond that I'm not really very sure which of several paths to take with this. We have discussed this topic several times on the Slack workspace, but didn't settle on anything yet. If you haven't already, maybe you could join the Slack workspace and post or DM me on there, and we can schedule some time to shake this down to some kind of decision and move things forward like that? Many thanks for raising the topic. I'm very interested in this, as are several others. |
Hey @bolbken just wanted to let you know I'm working on async stuff in this branch: https://github.com/johnbywater/eventsourcing/commits/feature/async_by_duplicating_methods |
Hi there, just wondering, is there an idea to proceed to implement the async approach? The ideas and the level of maturity are quite astonishing in this library, yet the sync-only nature unfortunately makes it pretty unusable for async-based projects I happen to work on, so I'm interested to hear any thoughts on that, thank you. |
As far as I can see from the branch commit discussion (f919f27#r71967500) the reason why the development has been abandoned (as far as I understand, at least) is that the async codebase performed poorly compared to the sync on in terms of speed and interested to understand whether it is possible to release those changes even considering this fact (maybe as some part of the beta/non-stable API). |
Hey @MrLokans thanks for your nice comments about the library, and for your interest in this topic. Let's look into this again. I'm definitely sympathetic about the concern you described. Perhaps this time we should firstly get it working with the POPO persistence module, and then see about other database adaptors. |
@johnbywater: Is this effort for the persistence mechanisms to support asyncio or also @event decorator will be able to work on async command methods. |
I don't think the aggregate decorator would need to be an async function? It's just the stack down to the database? That's unless the aggregate command methods are async. Are you writing aggregates with async command methods too? |
I'm attempting to use this library in an ASGI app and wondering if an asyncio effort is underway. Saw this issue comment
The principles and abstraction of this library are stellar! I'm hoping with an async implementation the IO performance could be stellar as well. Thoughts? Progress?
I'm eyeing the async SQLAlchemy functionality and/or asyncpg as orm/driver candidates.
High potential I could get some time to begin a naive async implementation soon.
The text was updated successfully, but these errors were encountered: