Skip to content

Bug/postgres jsonb processing#67

Merged
fivetran-catfritz merged 5 commits intomainfrom
bug/postgres-jsonb-processing
Mar 10, 2026
Merged

Bug/postgres jsonb processing#67
fivetran-catfritz merged 5 commits intomainfrom
bug/postgres-jsonb-processing

Conversation

@fivetran-catfritz
Copy link
Copy Markdown
Contributor

@fivetran-catfritz fivetran-catfritz commented Mar 10, 2026

PR Overview

Package version introduced in this PR:

  • v1.4.2

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

  • GA-1018822

Summary of changes:

  • Fixes PostgreSQL compatibility issue in get_url_tags_query() macro where jsonb columns were incorrectly processed using json_array_elements() instead of jsonb_array_elements(), causing database errors when url_tags column is stored as jsonb datatype.

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 Mar 10, 2026
@fivetran-catfritz fivetran-catfritz marked this pull request as ready for review March 10, 2026 19:30
@fivetran-catfritz fivetran-catfritz added the docs:ready Triggers the docs generator workflow. label Mar 10, 2026
Copy link
Copy Markdown
Contributor

@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

@fivetran-catfritz fivetran-catfritz merged commit 0ad3c9f into main Mar 10, 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