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 CI - pandas-gbq dependency issue #1768

Merged
merged 3 commits into from
Feb 16, 2023
Merged

Fix CI - pandas-gbq dependency issue #1768

merged 3 commits into from
Feb 16, 2023

Conversation

utkarsharma2
Copy link
Collaborator

@utkarsharma2 utkarsharma2 commented Feb 16, 2023

Description

What is the current behavior?

version: 0.13.2 of pandas-gbq is installed on CI containers causing it to fail.

  1. Exception not found
  2. pandas - 1.5.3 is not compatible 0.13.2

closes: #1761

What is the new behavior?

We found out that the snowflake-connector-python is using pyarrow which uses google packages and pandas-gbq uses the same google packages due to which the 0.13.2 of pandas was getting installed.

Does this introduce a breaking change?

Nope

Checklist

  • Created tests which fail without the change (if possible)
  • Extended the README / documentation, if necessary

@codecov
Copy link

codecov bot commented Feb 16, 2023

Codecov Report

Base: 93.20% // Head: 97.72% // Increases project coverage by +4.51% 🎉

Coverage data is based on head (3d4e4ca) compared to base (e09a88c).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1768      +/-   ##
==========================================
+ Coverage   93.20%   97.72%   +4.51%     
==========================================
  Files          91       21      -70     
  Lines        4845      835    -4010     
  Branches      479        0     -479     
==========================================
- Hits         4516      816    -3700     
+ Misses        238       19     -219     
+ Partials       91        0      -91     
Impacted Files Coverage Δ
python-sdk/src/astro/sql/__init__.py
...hon-sdk/src/astro/files/locations/google/gdrive.py
python-sdk/src/astro/databases/duckdb.py
python-sdk/src/astro/files/types/base.py
python-sdk/src/astro/databases/__init__.py
python-sdk/src/astro/exceptions.py
python-sdk/src/astro/settings.py
python-sdk/src/astro/databases/postgres.py
python-sdk/src/astro/custom_backend/serializer.py
python-sdk/src/astro/utils/dataframe.py
... and 60 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@utkarsharma2 utkarsharma2 changed the title Fix CI Fix CI - pandas-gbq dependency issue Feb 16, 2023
@utkarsharma2 utkarsharma2 marked this pull request as ready for review February 16, 2023 11:16
Copy link
Collaborator

@tatiana tatiana left a comment

Choose a reason for hiding this comment

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

Thanks, @utkarsharma2 ! Dependency hell 🙈
Happy for this to be merged once the checks pass!

@utkarsharma2
Copy link
Collaborator Author

The failing tests are not relevant to this PR. Going ahead and merging it.

@utkarsharma2 utkarsharma2 merged commit 6fb81eb into main Feb 16, 2023
@utkarsharma2 utkarsharma2 deleted the FixCISnowflake branch February 16, 2023 11:41
@utkarsharma2 utkarsharma2 mentioned this pull request Feb 16, 2023
2 tasks
utkarsharma2 added a commit that referenced this pull request Feb 21, 2023
# Description
## What is the current behavior?
version: 0.13.2 of pandas-gbq is installed on CI containers causing it
to fail.
1. Exception not found
2. pandas - 1.5.3 is not compatible 0.13.2

closes: #1761

## What is the new behavior?
We found out that the snowflake-connector-python is using pyarrow which
uses google packages and pandas-gbq uses the same google packages due to
which the 0.13.2 of pandas was getting installed.


## Does this introduce a breaking change?
Nope

### Checklist
- [ ] Created tests which fail without the change (if possible)
- [ ] Extended the README / documentation, if necessary
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.

CI Failure - pandas-gbq dependency issue
3 participants