Skip to content

Destination Bigquery: throw useful error when crashing in setup - #61018

Merged
Edward Gao (edgao) merged 7 commits into
masterfrom
edgao/avoid_race
Jun 4, 2025
Merged

Destination Bigquery: throw useful error when crashing in setup#61018
Edward Gao (edgao) merged 7 commits into
masterfrom
edgao/avoid_race

Conversation

@edgao

@edgao Edward Gao (edgao) commented May 30, 2025

Copy link
Copy Markdown
Contributor

closes https://github.com/airbytehq/airbyte-internal-issues/issues/13156 - see comments for explanation. tl;dr this solves a race condition, which was causing us to emit a trace message Input was fully read, but some streams did not receive a terminal stream status message instead of the real error.

@vercel

vercel Bot commented May 30, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
airbyte-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 3, 2025 10:18pm

@github-actions

Copy link
Copy Markdown
Contributor

👋 Greetings, Contributor!

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.
  • /run-cat-tests - Runs legacy CAT tests (Connector Acceptance Tests)

📝 Edit this welcome message.

@github-actions

github-actions Bot commented May 30, 2025

Copy link
Copy Markdown
Contributor

destination-bigquery Connector Test Results

178 tests   156 ✅  25m 41s ⏱️
 11 suites   22 💤
 11 files      0 ❌

Results for commit 6bad050.

♻️ This comment has been updated with latest results.

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:

# Conflicts:
#	docs/integrations/destinations/bigquery.md
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.

5 participants