Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update eff-sat branch from main #281

Merged
merged 79 commits into from
Nov 7, 2024
Merged

Conversation

tkiehn
Copy link
Collaborator

@tkiehn tkiehn commented Nov 7, 2024

Update dev-branch from main to include the new adapters

filcov and others added 30 commits August 29, 2024 10:33
Done for snowflake, biquery and exasol. need to checkout if there have been additional changes betweem the current main and the one this pull was done for
Then moving on to other adapters
Modified redshift, postgres and synapse NH link, now need to test w/ skeleton
change synapse default from VARCHAR to NVARCHAR(1000)
…tead-full-scan

run count instead of full scan for rec-trac-sats
Might need to ask Paul for the reason he done that...
…t (Postgres)

Need to ask Paul for his reason
…macro instead of directly accessing the variables
…-track-sat-to-use-string-default-dtype-macro

update fabric stage and rec_track_sat to use sring_default_dtype()
tderk and others added 29 commits October 22, 2024 09:10
change hardcoded default hash_datatype to VARCHAR2(40)
fix error ghost record for LONG-datatypes
fix ambigious endmacro
fix error messages in beginning and end of all times macros
…owflake-ghost-records

Add the datatypes Array and Variant to Snowflake ghost records
improve numeric handling of snowflake ghost records
improve handling of the various timestamp formats in snowflake ghost records
…foreign-hashkeys

140 optionality for nh link foreign hashkeys
…pported-timestamp_sub-function

FIX:databricks/postgres/redshift: replace unsupported timestamp_sub- and dateadd-functions in pit macros
@tkiehn tkiehn merged commit ab9dd5b into effectivity_satellites_other_adapters Nov 7, 2024
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.

5 participants