-
Notifications
You must be signed in to change notification settings - Fork 116
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
feature: Add snapshotting started message to ingestion process #1377
feature: Add snapshotting started message to ingestion process #1377
Conversation
@snork-alt Is this intended to be merged? There's still todo! in code. |
This PR is just for sending snapshotting started/done ingestion messages. I think logic of how it will be used should be implemented in a separate PR. |
I was worried the |
Got it. Anyway, I have changed it to simple comment https://github.com/getdozer/dozer/pull/1377/files#diff-dfd9b1553d77e73ce70cadd635b4f5e6a925b5c246d31a740975ff2d3e27ad30R214-R215 |
Oh I was looking at a outdated version of the PR. Sorry. |
No description provided.