Skip to content

Feature/redshift model large#129

Merged
fivetran-catfritz merged 16 commits intomainfrom
feature/redshift-model-large
Feb 19, 2026
Merged

Feature/redshift model large#129
fivetran-catfritz merged 16 commits intomainfrom
feature/redshift-model-large

Conversation

@fivetran-catfritz
Copy link
Copy Markdown
Contributor

@fivetran-catfritz fivetran-catfritz commented Feb 4, 2026

PR Overview

Package version introduced in this PR:

  • v0.11.1

This PR addresses the following Issue/Feature(s):

  • GA-1015680

Summary of changes:

  • Adds support for Redshift's SUPER data type in the sfdc_formula_view macro, enabling proper handling of large model configurations stored as JSON in the fivetran_formula_model.model_large column.

Submission Checklist

  • Alignment meeting with the reviewer (if needed)
    • Timeline and validation requirements discussed
  • Provide validation details:
    • Validation Steps: Check for unintentional effects (e.g., add/run consistency & integrity tests)
    • Testing Instructions: Confirm the change addresses the issue(s)
    • Focus Areas: Complex logic or queries that need extra attention
  • Merge any relevant open PRs into this PR

Changelog

  • Draft changelog for PR
  • Final changelog for release review

@fivetran-catfritz fivetran-catfritz self-assigned this Feb 12, 2026
@fivetran-catfritz fivetran-catfritz marked this pull request as ready for review February 12, 2026 23:27
@fivetran-catfritz fivetran-catfritz added the docs:ready Triggers the docs generator workflow. label Feb 12, 2026
@@ -1,3 +1,4 @@
{{ salesforce_formula_utils.sfdc_formula_view(
source_table = 'account',
using_quoted_identifiers = var('using_quoted_identifiers', false),
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wanted a way to test this.

Comment on lines +23 to +28
{%- set run_query %}
select 'has_values'
from {{ source(source_name, 'fivetran_formula_model') }}
where model_large is not null
limit 1
{%- endset %}
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I figured this was the least expensive query to check for non-null values since we don't have to scan the entire table, but open to suggestions.

Copy link
Copy Markdown
Collaborator

@fivetran-joemarkiewicz fivetran-joemarkiewicz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with one CHANGELOG suggestion

CHANGELOG.md Outdated
[PR #129](https://github.com/fivetran/dbt_salesforce_formula_utils/pull/129) includes the following updates:

## Feature Update
- Adds support for Redshift's `SUPER` data type in the `sfdc_formula_view` macro, enabling proper handling of large model configurations stored as JSON in the `fivetran_formula_model.model_large` column.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You mention it, but let's make this explicit that this entire feature update is only applicable to Redshift destinations.

@fivetran-catfritz fivetran-catfritz added docs:ready Triggers the docs generator workflow. and removed docs:ready Triggers the docs generator workflow. labels Feb 19, 2026
@fivetran-catfritz fivetran-catfritz merged commit 6411a75 into main Feb 19, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs:ready Triggers the docs generator workflow.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants