Skip to content

fix(main): Truncate sync time to microsecond#13484

Merged
kodiakhq[bot] merged 2 commits intomainfrom
fix/cli/truncate-sync-time
Aug 31, 2023
Merged

fix(main): Truncate sync time to microsecond#13484
kodiakhq[bot] merged 2 commits intomainfrom
fix/cli/truncate-sync-time

Conversation

@candiduslynx
Copy link
Copy Markdown
Contributor

@candiduslynx candiduslynx commented Aug 30, 2023

Fixes #13457
If we have a destination that supports timestamps with precisions higher than microsecond (like MSSQL), we may end up removing all previously written data in overwrite-delete-stale mode.

We already solved it once for Snowflake in #10354

@candiduslynx candiduslynx added the automerge Automatically merge once required checks pass label Aug 31, 2023
@candiduslynx candiduslynx self-assigned this Aug 31, 2023
@kodiakhq kodiakhq bot merged commit 8f874b6 into main Aug 31, 2023
@kodiakhq kodiakhq bot deleted the fix/cli/truncate-sync-time branch August 31, 2023 08:53
kodiakhq bot pushed a commit that referenced this pull request Aug 31, 2023
🤖 I have created a release *beep* *boop*
---


## [3.16.2](cli-v3.16.1...cli-v3.16.2) (2023-08-31)


### Bug Fixes

* **main:** Truncate sync time to microsecond ([#13484](#13484)) ([8f874b6](8f874b6))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge Automatically merge once required checks pass

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug: MSSQL destination truncates all records when using overwrite-delete-stale

3 participants