-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
Destinations: handle collisions between destination states #36768
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
Sweet. I'll put it in the queue after this #36620 |
lmk if you want me to wrap all the state stuff in try-catch return emptyMap(), I didn't add that to this pr but it's probably an easy change |
} else { | ||
OffsetDateTime.MIN | ||
} | ||
}.associate { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
neat!
479720c
to
6e15a4e
Compare
/publish-java-cdk
|
@gisripa this should solve the dest-postgres dup state key thing, though... probably not worth merging until after the cdk stabilizes
also wraps destination-state-related code in try-catch, since we treat state as a best-effort thing.