-
Notifications
You must be signed in to change notification settings - Fork 306
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
deps: make pyarrow and BQ Storage optional dependencies #1282
Conversation
🤖 I detect that the PR title and the commit message differ and there's only one commit. To use the PR title for the commit history, you can use Github's automerge feature with squashing, or use -- conventional-commit-lint bot |
@jonahgeorge @steffnay I've updated the PR with some adjustments to unit tests & a bit of synchronization between |
* update dependencies * deps: pyarrow extras * clean up comments * add test pyarrow skips * replace storage checks * update tests * update tests * Update setup.py * update system tests * update verify_pandas_imports * add pyarrow guards * add datetime check * change pyarrow import * update * add pyarrow skips * fix types * lint * Update google/cloud/bigquery/client.py Co-authored-by: Tim Swast <swast@google.com> * update pyarrow version * update test * lint * update pyarrow req * update noxfile * remove bignum check * remove comments * add test importorskip * update test * update test * update dependency * change version * update imports Co-authored-by: Anthonios Partheniou <partheniou@google.com> Co-authored-by: Tim Swast <swast@google.com>
BEGIN_COMMIT_OVERRIDE
deps: make pyarrow and BQ Storage optional dependencies
END_COMMIT_OVERRIDE
Restores optional checks for pyarrow and google-cloud-bigquery-storage, which were removed in deps!: BigQuery Storage and pyarrow are required dependencies #776
Restores guards against bad pyarrow versions: fix: guard imports against unsupported pyarrow versions #934
Adds pyarrow and google-cloud-bigquery-storage checks to verify_pandas_imports
Fixes #1196 🦕