-
Notifications
You must be signed in to change notification settings - Fork 163
make package compatible with Azure Synapse and Microsoft Fabric #529
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
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -298,7 +298,7 @@ | |
| {% endmacro -%} | ||
| {% macro sqlserver__get_invocations_dml_sql() -%} | ||
| {% macro fabric__get_invocations_dml_sql() -%} | ||
|
||
| {% set invocation_values %} | ||
| select | ||
| "1", | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -201,7 +201,7 @@ | |
| {%- endmacro %} | ||
|
|
||
|
|
||
| {% macro sqlserver__get_model_executions_dml_sql(models) -%} | ||
| {% macro fabric__get_model_executions_dml_sql(models) -%} | ||
|
||
| {% if models != [] %} | ||
| {% set model_execution_values %} | ||
| select | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -123,7 +123,7 @@ | |
| {%- endmacro %} | ||
|
|
||
|
|
||
| {% macro sqlserver__get_models_dml_sql(models) -%} | ||
| {% macro fabric__get_models_dml_sql(models) -%} | ||
|
||
|
|
||
| {% if models != [] %} | ||
| {% set model_values %} | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -214,7 +214,7 @@ | |
| {% endif %} | ||
| {% endmacro -%} | ||
|
|
||
| {% macro sqlserver__get_seed_executions_dml_sql(seeds) -%} | ||
| {% macro fabric__get_seed_executions_dml_sql(seeds) -%} | ||
|
||
| {% if seeds != [] %} | ||
| {% set seed_execution_values %} | ||
| select | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -107,7 +107,7 @@ | |
| {% endif %} | ||
| {%- endmacro %} | ||
|
|
||
| {% macro sqlserver__get_seeds_dml_sql(seeds) -%} | ||
| {% macro fabric__get_seeds_dml_sql(seeds) -%} | ||
|
||
|
|
||
| {% if seeds != [] %} | ||
| {% set seed_values %} | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -214,7 +214,7 @@ | |
| {% endif %} | ||
| {% endmacro -%} | ||
|
|
||
| {% macro sqlserver__get_snapshot_executions_dml_sql(snapshots) -%} | ||
| {% macro fabric__get_snapshot_executions_dml_sql(snapshots) -%} | ||
|
||
| {% if snapshots != [] %} | ||
| {% set snapshot_execution_values %} | ||
| select | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -118,7 +118,7 @@ | |
| {%- endmacro %} | ||
|
|
||
|
|
||
| {% macro sqlserver__get_snapshots_dml_sql(snapshots) -%} | ||
| {% macro fabric__get_snapshots_dml_sql(snapshots) -%} | ||
|
||
|
|
||
| {% if snapshots != [] %} | ||
| {% set snapshot_values %} | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -108,7 +108,7 @@ | |
| {%- endmacro %} | ||
|
|
||
|
|
||
| {% macro sqlserver__get_sources_dml_sql(sources) -%} | ||
| {% macro fabric__get_sources_dml_sql(sources) -%} | ||
|
||
|
|
||
| {% if sources != [] %} | ||
| {% set source_values %} | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -198,7 +198,7 @@ | |
| {% endif %} | ||
| {% endmacro -%} | ||
|
|
||
| {% macro sqlserver__get_test_executions_dml_sql(tests) -%} | ||
| {% macro fabric__get_test_executions_dml_sql(tests) -%} | ||
|
||
| {% if tests != [] %} | ||
| {% set test_execution_values %} | ||
| select | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -96,7 +96,7 @@ | |
| {%- endmacro %} | ||
|
|
||
|
|
||
| {% macro sqlserver__get_tests_dml_sql(tests) -%} | ||
| {% macro fabric__get_tests_dml_sql(tests) -%} | ||
|
||
|
|
||
| {% if tests != [] %} | ||
| {% set test_values %} | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -57,7 +57,7 @@ | |
|
|
||
| {%- endmacro %} | ||
|
|
||
| {% macro sqlserver__insert_into_metadata_table(relation, fields, content) -%} | ||
| {% macro fabric__insert_into_metadata_table(relation, fields, content) -%} | ||
|
||
|
|
||
| {% set insert_into_table_query %} | ||
| insert into {{ relation }} {{ fields }} | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Replacing the sqlserver__ macro with fabric__ means SQL Server users will lose access to this specific implementation. Consider keeping both macros or creating a shared implementation that all three target types can use.