Skip to content

Destination snowflake: component tests - #68125

Merged
Edward Gao (edgao) merged 9 commits into
masterfrom
edgao/snowflake_component_tests
Oct 28, 2025
Merged

Edward Gao (edgao) merged 9 commits into
masterfrom
edgao/snowflake_component_tests

Conversation

@edgao

@edgao Edward Gao (edgao) commented Oct 15, 2025 •

Copy link
Copy Markdown
Contributor

What

The connectors' half of part of 1 of https://github.com/airbytehq/airbyte-internal-issues/issues/14865. Branched from #68167. Implement the new test interfaces in dest-snowflake + add (most) of the tests.

How

Just do it. (also bump dest-clickhouse to local cdk, to verify I didn't break anything)

Can this PR be safely reverted and rolled back?

  • YES 💚
  • NO ❌

Important

Auto-merge enabled.

This PR is set to merge automatically when all requirements are met.

@github-actions

Copy link
Copy Markdown
Contributor

👋 Greetings, Airbyte Team Member!

Here are some helpful tips and reminders for your convenience.

Helpful Resources

PR Slash Commands

Airbyte Maintainers (that's you!) can execute the following slash commands on your PR:

  • /format-fix - Fixes most formatting issues.
  • /bump-version - Bumps connector versions.
    • You can specify a custom changelog by passing changelog. Example: /bump-version changelog="My cool update"
    • Leaving the changelog arg blank will auto-populate the changelog from the PR title.
  • /run-cat-tests - Runs legacy CAT tests (Connector Acceptance Tests)
  • /build-connector-images - Builds and publishes a pre-release docker image for the modified connector(s).
  • JVM connectors:
    • /update-connector-cdk-version connector=<CONNECTOR_NAME> - Updates the specified connector to the latest CDK version.
      Example: /update-connector-cdk-version connector=destination-bigquery
    • /bump-bulk-cdk-version type=patch changelog='foo' - Bump the Bulk CDK's version. type can be major/minor/patch.
  • Python connectors:
    • /poe connector source-example lock - Run the Poe lock task on the source-example connector, committing the results back to the branch.
    • /poe source example lock - Alias for /poe connector source-example lock.
    • /poe source example use-cdk-branch my/branch - Pin the source-example CDK reference to the branch name specified.
    • /poe source example use-cdk-latest - Update the source-example CDK dependency to the latest available version.

📝 Edit this welcome message.

@github-actions

github-actions Bot commented Oct 15, 2025 •

Copy link
Copy Markdown
Contributor

destination-snowflake Connector Test Results

 27 files   27 suites   22m 1s ⏱️
329 tests 293 ✅ 36 💤 0 ❌
333 runs  297 ✅ 36 💤 0 ❌

Results for commit 5f69b52.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Oct 15, 2025 •

Copy link
Copy Markdown
Contributor

@github-actions

github-actions Bot commented Oct 15, 2025 •

Copy link
Copy Markdown
Contributor

destination-clickhouse Connector Test Results

 14 files   14 suites   22m 50s ⏱️
283 tests 234 ✅ 49 💤 0 ❌
313 runs  264 ✅ 49 💤 0 ❌

Results for commit 5f69b52.

♻️ This comment has been updated with latest results.

@tryangul
Ryan Br... (tryangul) force-pushed the rbroughan/component-tests-operations-test-1 branch from 57e52d2 to f967fcc Compare October 15, 2025 23:24
Base automatically changed from rbroughan/component-tests-operations-test-1 to master October 16, 2025 18:19
@edgao
Edward Gao (edgao) force-pushed the edgao/snowflake_component_tests branch from 391845e to efec600 Compare October 17, 2025 18:12
@edgao
Edward Gao (edgao) changed the base branch from master to edgao/component_test_cdk_stuff October 17, 2025 18:12
@edgao
Edward Gao (edgao) force-pushed the edgao/component_test_cdk_stuff branch from dd20b70 to 649d28a Compare October 17, 2025 18:14
@edgao
Edward Gao (edgao) force-pushed the edgao/snowflake_component_tests branch 2 times, most recently from 3e723d8 to e8faa99 Compare October 17, 2025 18:19
@edgao
Edward Gao (edgao) force-pushed the edgao/component_test_cdk_stuff branch 2 times, most recently from 1500032 to 1c3eb77 Compare October 17, 2025 19:41
@edgao
Edward Gao (edgao) force-pushed the edgao/snowflake_component_tests branch from e8faa99 to c572e8a Compare October 17, 2025 19:41
@edgao
Edward Gao (edgao) force-pushed the edgao/component_test_cdk_stuff branch from 1c3eb77 to 8761488 Compare October 17, 2025 20:00
@edgao
Edward Gao (edgao) force-pushed the edgao/snowflake_component_tests branch from c572e8a to 2a3d816 Compare October 17, 2025 20:00
@edgao
Edward Gao (edgao) force-pushed the edgao/snowflake_component_tests branch from 2a3d816 to a9bf914 Compare October 17, 2025 20:01
@edgao
Edward Gao (edgao) force-pushed the edgao/component_test_cdk_stuff branch from 8761488 to 5e87d52 Compare October 17, 2025 20:01
@edgao
Edward Gao (edgao) force-pushed the edgao/snowflake_component_tests branch from a9bf914 to f2d927e Compare October 17, 2025 20:06
connection.prepareStatement(
"""
/**
* TEST ONLY. We have a much more performant implementation in

@edgao Edward Gao (edgao) Oct 17, 2025 •

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Increasingly convinced we should split this into a separate TestTableOps interface. Will do that in https://github.com/airbytehq/airbyte-internal-issues/issues/14912

@edgao
Edward Gao (edgao) marked this pull request as ready for review October 17, 2025 20:12
@edgao
Edward Gao (edgao) requested a review from a team as a code owner October 17, 2025 20:12
@@ -1 +1 @@
cdkVersion=0.1.52
cdkVersion=local

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.

I assume that you don't want to commit this change.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

yep, PR checks will block this anyway. This is just to get CI to run against the CDK changes

@@ -1,3 +1,3 @@
testExecutionConcurrency=-1
cdkVersion=0.1.52
cdkVersion=local

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.

Same 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.

:shipit: assuming confirmation or revert of the cdk version changes in this PR.

Base automatically changed from edgao/component_test_cdk_stuff to master October 28, 2025 15:41
# Conflicts:
#	airbyte-integrations/connectors/destination-clickhouse/gradle.properties
#	airbyte-integrations/connectors/destination-snowflake/gradle.properties
#	airbyte-integrations/connectors/destination-snowflake/src/main/kotlin/io/airbyte/integrations/destination/snowflake/client/SnowflakeAirbyteClient.kt
@edgao
Edward Gao (edgao) enabled auto-merge (squash) October 28, 2025 17:03
@github-actions

Copy link
Copy Markdown
Contributor

Deploy preview for airbyte-docs ready!

✅ Preview
https://airbyte-docs-43ffmxr69-airbyte-growth.vercel.app

Built with commit 5f69b52.
This pull request is being automatically deployed with vercel-action

@edgao
Edward Gao (edgao) merged commit a8eb285 into master Oct 28, 2025
41 checks passed
@edgao
Edward Gao (edgao) deleted the edgao/snowflake_component_tests branch October 28, 2025 17:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants