Skip to content

Differentiate between inserts and updates in flows#2846

Merged
gbrodman merged 1 commit into
google:masterfrom
gbrodman:flowInserts
Oct 13, 2025
Merged

Differentiate between inserts and updates in flows#2846
gbrodman merged 1 commit into
google:masterfrom
gbrodman:flowInserts

Conversation

@gbrodman

@gbrodman gbrodman commented Oct 10, 2025

Copy link
Copy Markdown
Collaborator

Updates (AKA merges) run an extra SELECT statement to figure out if the resource exists so that it can merge the entity into the existing object in Hibernate's schema. When we're inserting new rows (such as new poll messages or resource creates), we know that we don't need to do that merge. Doing this should save us some SELECT statements (this has borne out to be the truth in alpha)


This change is Reviewable

Updates (AKA merges) run an extra SELECT statement to figure out if the
resource exists so that it can merge the entity into the existing object
in Hibernate's schema. When we're inserting new rows (such as new poll
messages or resource creates), we know that we don't need to do that
merge. Doing this should save us some SELECT statements (this has borne
out to be the truth in alpha)
@gbrodman
gbrodman requested a review from weiminyu October 13, 2025 14:05

@weiminyu weiminyu left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@weiminyu reviewed 15 of 15 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @gbrodman)

@gbrodman
gbrodman added this pull request to the merge queue Oct 13, 2025
Merged via the queue into google:master with commit 5faf3d2 Oct 13, 2025
9 checks passed
@gbrodman
gbrodman deleted the flowInserts branch October 13, 2025 16:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants