-
Notifications
You must be signed in to change notification settings - Fork 79
SNOW-2464939 Update snowflake-snowpark-python version to 1.41.0 #2672
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
base: main
Are you sure you want to change the base?
Conversation
|
Can we use connector ver. |
|
BTW, see my feature request open here #2679 happy to help on this front as it is somewhat blocking some features on my end. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sfc-gh-daniszewski please review patterns such as
snowflake-cli/tests_integration/test_stage.py
Line 338 in b0306e1
| sys.version_info >= (3, 12), reason="Snowpark is not supported in Python >= 3.12" |
| "setuptools==80.8.0", | ||
| "snowflake-connector-python[secure-local-storage]==3.18.0", | ||
| 'snowflake-snowpark-python==1.33.0;python_version < "3.12"', | ||
| 'snowflake-snowpark-python==1.41.0', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am curious if such strict dependency requirements are necessary. What exactly are the API requirements from the snowpark package? I feel like setting up a tox matrix for supported versions should be a fairly easy way to relax the dependency on a specific version if there are no breaking API changes.
Pre-review checklist
Changes description
...