Skip to content

Make sure views use non-resumable partitions - #69754

Merged
Wenqi Hu (burakku) merged 1 commit into
masterfrom
wenqi-mssql-rollout-fix-9
Nov 19, 2025
Merged

Make sure views use non-resumable partitions#69754
Wenqi Hu (burakku) merged 1 commit into
masterfrom
wenqi-mssql-rollout-fix-9

Conversation

@burakku

@burakku Wenqi Hu (burakku) commented Nov 19, 2025

Copy link
Copy Markdown
Contributor

What

We try to avoid TABLESAMPLE on views

How

Review guide

User Impact

Can this PR be safely reverted and rolled back?

  • YES 💚
  • NO ❌

@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 bump=patch changelog='foo' - Bump the Bulk CDK's version. bump 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 Nov 19, 2025

Copy link
Copy Markdown
Contributor

source-mssql Connector Test Results

 10 files   10 suites   45s ⏱️
 94 tests  94 ✅ 0 💤 0 ❌
204 runs  204 ✅ 0 💤 0 ❌

Results for commit 1c584a7.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Nov 19, 2025

Copy link
Copy Markdown
Contributor

Note

Detected that there are differences in the Gradle dependencies.

@github-actions

github-actions Bot commented Nov 19, 2025

Copy link
Copy Markdown
Contributor

Deploy preview for airbyte-docs ready!

✅ Preview
https://airbyte-docs-3ohvofzd1-airbyte-growth.vercel.app

Built with commit 1c584a7.
This pull request is being automatically deployed with vercel-action

@burakku
Wenqi Hu (burakku) force-pushed the wenqi-mssql-rollout-fix-9 branch 2 times, most recently from 7213310 to 5499593 Compare November 19, 2025 22:31
@burakku Wenqi Hu (burakku) changed the title Update CDK version Make sure views use non-resumable partitions Nov 19, 2025
@burakku
Wenqi Hu (burakku) merged commit e5d3164 into master Nov 19, 2025
37 checks passed
@burakku
Wenqi Hu (burakku) deleted the wenqi-mssql-rollout-fix-9 branch November 19, 2025 23:03
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