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

Snwoflake v2.4.0 #24

Merged
merged 55 commits into from
Sep 30, 2024
Merged

Snwoflake v2.4.0 #24

merged 55 commits into from
Sep 30, 2024

Conversation

wrehman-skap
Copy link

Overview

SNOW-XXXXX

Pre-review checklist

  • This change should be part of a Behavior Change Release. See go/behavior-change.
  • This change has passed Merge gate tests
  • Snowpipe Changes
  • Snowpipe Streaming Changes
  • This change is TEST-ONLY
  • This change is README/Javadocs only
  • This change is protected by a config parameter <PARAMETER_NAME> eg snowflake.ingestion.method.
    • Yes - Added end to end and Unit Tests.
    • No - Suggest why it is not param protected
  • Is his change protected by parameter <PARAMETER_NAME> on the server side?
    • The parameter/feature is not yet active in production (partial rollout or PrPr, see Changes for Unreleased Features and Fixes).
    • If there is an issue, it can be safely mitigated by turning the parameter off. This is also verified by a test (See go/ppp).

sfc-gh-krosinski and others added 30 commits April 25, 2024 10:50
…wflakedb#824)

Building and unit testing process remains the same (Java8) but e2e testing is run on three different versions of JAVA: 8, 11 and 17. This means that there are 3 times more testing jobs, but they all run in parallel, hence total run time should remain ~the same.
…sions (snowflakedb#826)

Building and unit testing process remains the same (Java8) but e2e testing is run on three different versions of JAVA: 8, 11 and 17. This means that there are 3 times more testing jobs, but they all run in parallel, hence total run time should remain ~the same.
…JRE versions (snowflakedb#827)

Building and unit testing process remains the same (Java8) but stress testing is run on three different versions of JAVA: 8, 11 and 17. This means that there are 3 times more testing jobs, but they all run in parallel, hence total run time should remain ~the same.
sfc-gh-gjachimko and others added 23 commits July 18, 2024 10:45
Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Michał Bobowski <[email protected]>
Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Michał Bobowski <[email protected]>
…kedb#863)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Michał Bobowski <[email protected]>
…7.0 (snowflakedb#857)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Michał Bobowski <[email protected]>
# Conflicts:
#	.github/workflows/End2EndTestApache.yml
#	.github/workflows/End2EndTestConfluent.yml
#	.github/workflows/StressTest.yml
#	scripts/process_licenses.py
#	src/main/java/com/snowflake/kafka/connector/internal/streaming/SchematizationUtils.java
#	src/main/java/com/snowflake/kafka/connector/internal/streaming/SnowflakeSinkServiceV2.java
#	src/main/java/com/snowflake/kafka/connector/records/RecordService.java
#	src/test/java/com/snowflake/kafka/connector/internal/streaming/SnowflakeSinkServiceV2IT.java
#	src/test/java/com/snowflake/kafka/connector/internal/streaming/TopicPartitionChannelIT.java
Copy link

@JonathonO JonathonO left a comment

Choose a reason for hiding this comment

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

@wrehman-skap

I assume our Snowflake Connector wrapper needed to be updated to 2.4.0 also.

@wrehman-skap
Copy link
Author

@JonathonO Yes, we have deployed it for all connector, tenant-metadata is only using plugin-default entry for all.
I will extract wrapper branch after excluding auto create schema commits.

@JonathonO JonathonO self-requested a review September 30, 2024 08:24
@wrehman-skap wrehman-skap merged commit 004d251 into streamkap-main Sep 30, 2024
0 of 144 checks passed
@JonathonO JonathonO deleted the snwoflake-v2.4.0 branch September 30, 2024 10:42
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.

10 participants