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

fix(bigquery-firestore-export): avoid the setup getting stuck #426

Merged
merged 8 commits into from
May 4, 2024

Conversation

jauntybrain
Copy link
Collaborator

Fixes a bug where the extension setup process gets stuck if the transfer config collection is provided, transfer config name is found, but the config itself does not exist.

Resolves #411.

cabljac and others added 6 commits February 21, 2024 11:00
@pr-Mais
Copy link
Collaborator

pr-Mais commented Mar 26, 2024

Can we add tests

@jauntybrain
Copy link
Collaborator Author

@pr-Mais added a new test

Copy link
Collaborator

@cabljac cabljac left a comment

Choose a reason for hiding this comment

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

lgtm!

@jauntybrain jauntybrain merged commit ced77b2 into next May 4, 2024
8 checks passed
@jauntybrain jauntybrain deleted the @invertase/bq-fs-export-transfer-config branch May 4, 2024 07:31
pr-Mais pushed a commit that referenced this pull request May 8, 2024
* docs(*): add gemini docs on regions

* fix(firestore-semantic-search): neighbor_count parameter location (#391)

* fix(firestore-semantic-search): neighbor_count parameter incorrect location

* feat(firestore-multimodal-genai): add callable function (#422)

* feat(firestore-multimodal-genai): add callable function

* refactor(firestore-multimodal-genai): tidy some code up a little

* docs: remove reference to VARIABLE_FIELDS (#421)

* chore(firestore-multimodal-genai): bump extension version

* fix(bigquery-firestore-export): avoid the setup getting stuck if the transfer config is not found

* feat(bigquery-firestore-export): transferConfig not found test

---------

Co-authored-by: Jacob Cable <[email protected]>
Co-authored-by: Jacob Cable <[email protected]>
@flyingcircle
Copy link

Is there a release scheduled with this fix?

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.

bigquery-firestore-export - unable to create transferConfig
4 participants