Skip to content

Release/26.7.15#360

Closed
Devesh-Skyflow wants to merge 1 commit into
v3from
release/26.7.15
Closed

Release/26.7.15#360
Devesh-Skyflow wants to merge 1 commit into
v3from
release/26.7.15

Conversation

@Devesh-Skyflow

Copy link
Copy Markdown
Collaborator

Start with a concise summary of the PR. The first three sections are required. The questions present in each section is there to help you guide you what to add. They are meant to be overwritten by your comments.

Why

  • Why are you making the change?
  • What is the underlying issue that you are trying to case, in case of fix?
  • Why is it needed by the feature you are working on?
  • What is the intent behind making the change?

Goal

  • What is the intended outcome?
  • What part of the feature should start working?
  • What are the non-goals or will be covered in future PR?

Testing

  • How was the code tested?
  • If you haven't written unit tests, why?
  • What more testing is needed? Do you intend to manually test it after deployment?
  • Do you have any concerns if this changed is released to prod?

Tech debt

  • Is the PR adding to tech debt in any way?
  • Are you addressing some Tech debt in this PR?
  • If both the above are false, feel free to remove this section.

…ror-message cleanup) (#346)

Flow-DB now defaults record-level upsert to UPDATE (was REPLACE) and owns
upsert-placement validation server-side. The SDK already serializes
upsertType correctly and omits updateType when unset, so this is docs,
tests, and validation hygiene only.

- README: document upsertType is optional, default is now UPDATE, and
  UPDATE vs REPLACE semantics.
- Samples: drop redundant explicit upsertType(UPDATE) (now the default)
  in BulkMultiTableInsert{Sync,Async}; keep REPLACE samples as-is.
- Validations: defer upsert-placement checks to the backend (authoritative
  messages); SDK only guards empty upsert columns.
- Tests: cover default-UPDATE omission at request/record level, both-level
  serialization, and updated placement tests to reflect deferral.

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@Devesh-Skyflow
Devesh-Skyflow changed the base branch from main to v3 July 20, 2026 07:15
@Devesh-Skyflow
Devesh-Skyflow deleted the release/26.7.15 branch July 20, 2026 07:22
@Devesh-Skyflow

Copy link
Copy Markdown
Collaborator Author

This PR was auto-closed by GitHub when the head branch was renamed release/26.7.15v3-release/26.7.15. Continued in #361 (same contents).

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