BD-5772: Fix CDI UPDATED_AT recurring sync example#12394
Open
atom-evens wants to merge 1 commit intodevelopfrom
Open
BD-5772: Fix CDI UPDATED_AT recurring sync example#12394atom-evens wants to merge 1 commit intodevelopfrom
atom-evens wants to merge 1 commit intodevelopfrom
Conversation
The recurring sync example incorrectly stated that no rows would be
re-synced when a row's UPDATED_AT equals the stored timestamp. This
contradicted the documented behavior ("equal to or later than") and
the inclusive boundary warning elsewhere on the same page.
Made-with: Cursor
Contributor
There was a problem hiding this comment.
Pull request overview
Fixes a contradiction in the Cloud Data Ingestion best practices doc by updating the recurring sync example to match the stated inclusive UPDATED_AT boundary behavior.
Changes:
- Corrects the “second run” narrative to reflect that rows with
UPDATED_ATequal to the stored timestamp are re-synced. - Corrects the “third run” narrative to reflect that two rows sync (newer + equal-to-stored).
- Adds an in-page cross-reference to the section explaining duplicate sync behavior caused by inclusive boundaries.
varunkvv
approved these changes
Mar 6, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
A customer reported a contradiction in the Cloud Data Ingestion Best Practices page. The doc states that Braze syncs rows where
UPDATED_ATis "equal to or later than" the stored timestamp (inclusive boundary), and a separate section on the same page explicitly warns about duplicate data resulting from this inclusive behavior. However, the recurring sync example contradicted both of these by:customer_5678has anUPDATED_ATequal to the stored timestamp and should be re-synced per the inclusive boundary rule.UPDATED_ATequal to the stored timestamp).Changes
customer_9012row won't sync (its timestamp is earlier), but the existingcustomer_5678row is re-synced because itsUPDATED_ATequals the stored timestamp. Adds a cross-reference to the "Make sure the UPDATED_AT time isn't the same time as your sync" section.customer_1234row (later than stored) and the existingcustomer_5678row (equal to stored).Test plan
Made with Cursor