-
-
Notifications
You must be signed in to change notification settings - Fork 445
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
Snapshot upgrader need metadata #919
Comments
Actually I can't use IEventStore... I don't have the aggregate's id ! |
Hello there! We hope you are doing well. We noticed that this issue has not seen any activity in the past 90 days. If you still require assistance with this issue, please feel free to reopen it or create a new issue. Thank you for your understanding and cooperation. Best regards, |
Hello there! We hope you are doing well. We noticed that this issue has not seen any activity in the past 90 days. If you still require assistance with this issue, please feel free to reopen it or create a new issue. Thank you for your understanding and cooperation. Best regards, |
Hello there 👋 Long-time lurker here. Decided that it's time to contribute a bit to the repository. The underlying idea of enabling the event store to load domain events within a given sequence-number range makes sense to me. I would go ahead and prepare a PR if still relevant. Regarding your snapshot upgrading question, would it help if the specific snapshots in your application would carry their respective aggregate Id in the payload? That way you would have access to it when upgrading from one snapshot version to the other, no? Cheers |
Issue #919: Enable IEventStore to load events up to a given sequence number
My SnapshotUpgrader need to retrieve data from the aggregate's events.
For this, it use the IEventStore instance.
But it should only take into account events whose sequence is less than or equal to that of the snapshot.
This information should be an argument of the UpgradeAsync method.
Moreover IEventStore should provide a method to get a range of events.
The text was updated successfully, but these errors were encountered: