Skip to content

Conversation

@notatallshaw
Copy link
Contributor

See #53306 and CC: @potiuk

The issue is many versions (e.g. 1.33.0) of snowflake-snowpark-python require protobuf<6,>=3.20 but this conflicts largely with grpcio-status where many versions (e.g. 1.73.1) require protobuf<7.0.0,>=6.30.0.

This takes advantage of the fact that pip uses an upper bound as an indicator that that requirement must be resolved first, without adding any actual restrictions on upper bounds: https://pip.pypa.io/en/stable/topics/more-dependency-resolution/#the-resolver-algorithm

Unfortunately this particular one trips up even the optimistic back tracker optimization coming in pip 25.2, so either I need to figure out some more optimizations for pip's resolver to handle this, or pip needs a whole new resolver.

@boring-cyborg boring-cyborg bot added area:providers provider:snowflake Issues related to Snowflake provider labels Jul 13, 2025
@gopidesupavan gopidesupavan added the full tests needed We need to run full set of tests for this PR to merge label Jul 14, 2025
@gopidesupavan
Copy link
Member

cool @notatallshaw thanks :)

@gopidesupavan
Copy link
Member

let me close and reopen PR with full tests

@gopidesupavan
Copy link
Member

Triggered with full tests.

@potiuk
Copy link
Member

potiuk commented Jul 14, 2025

Oh. Cool. Let's just merge it - currently the "push" step will not happen even if we do full tests - it is only run in canary builds and that requires maintainer to run PR from Airflow repo.

@potiuk potiuk merged commit 4cc8ebf into apache:main Jul 14, 2025
179 checks passed
@potiuk
Copy link
Member

potiuk commented Jul 14, 2025

Thanks a lot @notatallshaw !. I am triggering canary build now.

@bugraoz93
Copy link
Contributor

Thanks a lot @notatallshaw !

@gopidesupavan
Copy link
Member

Nice the magic worked 😄

@potiuk
Copy link
Member

potiuk commented Jul 14, 2025

thanks @notatallshaw !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:providers full tests needed We need to run full set of tests for this PR to merge provider:snowflake Issues related to Snowflake provider

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants