Skip to content

Conversation

tkiehn
Copy link
Collaborator

@tkiehn tkiehn commented Oct 25, 2024

Description

improve handling of the various timestamp formats in snowflake ghost records
also improve numeric data type handling for ghost records

improve numeric handling of snowflake ghost records
@tkiehn tkiehn added the testing To trigger the automated test workflow as internal User. label Oct 25, 2024
@tkiehn tkiehn requested a review from tkirschke October 25, 2024 09:21
@remoteworkflow
Copy link

Link to workflow summary: https://github.com/ScalefreeCOM/datavault4dbt-ci-cd/actions/runs/11515494813


RESULTS for Synapse:
✅ dbt-tests
✅ dbt-macro-tests


RESULTS for Postgres:
✅ dbt-tests
✅ dbt-macro-tests


RESULTS for BigQuery:
✅ dbt-tests
✅ dbt-macro-tests


RESULTS for Redshift:
✅ dbt-tests
✅ dbt-macro-tests


RESULTS for Snowflake:
✅ dbt-tests
✅ dbt-macro-tests


RESULTS for Exasol:
❌ dbt-tests
✅ dbt-macro-tests


RESULTS for Fabric:
✅ dbt-tests
✅ dbt-macro-tests


RESULTS for Oracle:
✅ dbt-tests
✅ dbt-macro-tests


RESULTS for Databricks:
✅ dbt-tests
✅ dbt-macro-tests

@remoteworkflow remoteworkflow bot removed the testing To trigger the automated test workflow as internal User. label Oct 25, 2024
@tkiehn
Copy link
Collaborator Author

tkiehn commented Oct 28, 2024

@JPLangner just notified me that after the clocks were set one hour behind this weekend there are 2 rows for ghostrecords with different offsets.

We have to figure out how to handle this nicely, if at all

@tkirschke
Copy link
Member

@JPLangner just notified me that after the clocks were set one hour behind this weekend there are 2 rows for ghostrecords with different offsets.

We have to figure out how to handle this nicely, if at all

Please create new Issue for that

@tkirschke tkirschke merged commit a80a51b into main Oct 30, 2024
1 check passed
@tkirschke tkirschke deleted the improve-snowflake-ghostrecords branch October 30, 2024 12:09
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