Skip to content

bug/string-limit-bq-sf#193

Merged
fivetran-catfritz merged 9 commits into
mainfrom
bug/string-limit-bq-sf
Jun 24, 2026
Merged

bug/string-limit-bq-sf#193
fivetran-catfritz merged 9 commits into
mainfrom
bug/string-limit-bq-sf

Conversation

@fivetran-catfritz

@fivetran-catfritz fivetran-catfritz commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

PR Overview

Package version introduced in this PR:

  • v1.10.1

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

  • GA-1027686

Summary of changes:

  • ixes a runtime error on Snowflake, BigQuery, and Redshift that occurred when an issue's concatenated comments exceeded the warehouse's string length limit.

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 Jun 18, 2026

@fivetran-savage fivetran-savage left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Just need to revert the schema name in the integration_tests project.yml and then I can approve.

Comment thread integration_tests/dbt_project.yml Outdated
@fivetran-savage

fivetran-savage commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

You'll also need to resolve the merge conflicts from the releases we did today.

@fivetran-catfritz fivetran-catfritz added the docs:ready Triggers the docs generator workflow. label Jun 24, 2026

@fivetran-savage fivetran-savage left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@fivetran-catfritz fivetran-catfritz merged commit 6b7d333 into main Jun 24, 2026
9 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