Skip to content

runtime: add materialize-azure-fabric-warehouse to ser_policy #2048

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

Merged
merged 1 commit into from
Apr 6, 2025

Conversation

williamhbaker
Copy link
Member

@williamhbaker williamhbaker commented Apr 6, 2025

Description:

Azure Fabric Warehouse has a 1MB size limit on VARCHAR(MAX) columns, so we need to try to stay below that size for any individual string field, and probably more importantly the entire document, since the document is stored in such a column.

Workflow steps:

(How does one use this feature, and how has it changed)

Documentation links affected:

(list any documentation links that you created, or existing ones that you've identified as needing updates, along with a brief description)

Notes for reviewers:

(anything that might help someone review this PR)


This change is Reviewable

Azure Fabric Warehouse has a 1MB size limit on VARCHAR(MAX) columns, so we need
to try to stay below that size for any individual string field, and probably
more importantly the entire document, since the document is stored in such a
column.
@williamhbaker williamhbaker requested a review from psFried April 6, 2025 01:19
Copy link
Member

@dyaffe dyaffe left a comment

Choose a reason for hiding this comment

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

LGTM

@williamhbaker williamhbaker merged commit 6a53136 into master Apr 6, 2025
3 checks passed
@williamhbaker williamhbaker deleted the wb/fabric-ser-policy branch April 6, 2025 16:27
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.

2 participants