Skip to content
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

NO-SNOW: Preserve the old data type that goes into an ARRAY column for Schematization #730

Merged
merged 34 commits into from
Nov 3, 2023

Conversation

sfc-gh-tzhang
Copy link
Contributor

@sfc-gh-tzhang sfc-gh-tzhang commented Oct 20, 2023

Before this change, every element in the array will be added as a STRING, this change preserves the old data type in the source, for example when the input is [1, 2], the ingested value will be [1, 2] now instead of ["1", "2"]

Forked from #727, with additional tests

@sfc-gh-xhuang
Copy link
Collaborator

#711 is also good to fork and merge

@sfc-gh-azagrebin
Copy link

@sfc-gh-tzhang
is it something related to #724?

@sfc-gh-tzhang
Copy link
Contributor Author

#711 is also good to fork and merge

that should be a different PR? I don't think we have verified whether that will be a behavior change or not

@sfc-gh-tzhang
Copy link
Contributor Author

@sfc-gh-tzhang is it something related to #724?

Yes

@cchandurkar
Copy link

Hey any estimates on merging this? Will this be available in next release?

Copy link
Collaborator

@sfc-gh-japatel sfc-gh-japatel left a comment

Choose a reason for hiding this comment

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

lgtm! thanks

@sfc-gh-tzhang sfc-gh-tzhang merged commit bf2a11a into master Nov 3, 2023
32 checks passed
@sfc-gh-tzhang sfc-gh-tzhang deleted the tzhang-si-fix branch November 3, 2023 18:12
khsoneji pushed a commit to confluentinc/snowflake-kafka-connector that referenced this pull request Dec 4, 2023
…r Schematization (snowflakedb#730)

Before this change, every element in the array will be added as a STRING, this change preserves the old data type in the source, for example when the input is [1, 2], the ingested value will be [1, 2] now instead of ["1", "2"]

Forked from snowflakedb#727, with additional tests
khsoneji pushed a commit to confluentinc/snowflake-kafka-connector that referenced this pull request Dec 4, 2023
…r Schematization (snowflakedb#730)

Before this change, every element in the array will be added as a STRING, this change preserves the old data type in the source, for example when the input is [1, 2], the ingested value will be [1, 2] now instead of ["1", "2"]

Forked from snowflakedb#727, with additional tests
khsoneji pushed a commit to confluentinc/snowflake-kafka-connector that referenced this pull request Dec 4, 2023
…r Schematization (snowflakedb#730)

Before this change, every element in the array will be added as a STRING, this change preserves the old data type in the source, for example when the input is [1, 2], the ingested value will be [1, 2] now instead of ["1", "2"]

Forked from snowflakedb#727, with additional tests
EduardHantig pushed a commit to streamkap-com/snowflake-kafka-connector that referenced this pull request Feb 1, 2024
…r Schematization (snowflakedb#730)

Before this change, every element in the array will be added as a STRING, this change preserves the old data type in the source, for example when the input is [1, 2], the ingested value will be [1, 2] now instead of ["1", "2"]

Forked from snowflakedb#727, with additional tests
sudeshwasnik pushed a commit to confluentinc/snowflake-kafka-connector that referenced this pull request Feb 16, 2024
…r Schematization (snowflakedb#730)

Before this change, every element in the array will be added as a STRING, this change preserves the old data type in the source, for example when the input is [1, 2], the ingested value will be [1, 2] now instead of ["1", "2"]

Forked from snowflakedb#727, with additional tests
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.

6 participants