Skip to content

Commit

Permalink
Merge gate fix (#1282)
Browse files Browse the repository at this point in the history
Description

Our merge gate is broken because of a new python-dateutil release.

To fix merge gate, there are two changes

- We don't check python-dateutil is in package set. The reason is
that the latest version of python-dateutil is not on Snowflake yet,
and as a result, we use custom package and not specify this package
in session.
- We add session.custom_package_usage_config for the python-dateutil
related doc test.

Testing

Merge gate passing
  • Loading branch information
sfc-gh-sfan authored Mar 1, 2024
1 parent 270560b commit 6fb2515
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 1 deletion.
1 change: 1 addition & 0 deletions src/snowflake/snowpark/session.py
Original file line number Diff line number Diff line change
Expand Up @@ -918,6 +918,7 @@ def add_packages(
>>> # add numpy with the latest version on Snowflake Anaconda
>>> # and pandas with the version "1.3.*"
>>> # and dateutil with the local version in your environment
>>> session.custom_package_usage_config = {"enabled": True} # This is added because latest dateutil is not in snowflake yet
>>> session.add_packages("numpy", "pandas==1.5.*", dateutil)
>>> @udf
... def get_package_name_udf() -> list:
Expand Down
1 change: 0 additions & 1 deletion tests/integ/test_packaging.py
Original file line number Diff line number Diff line change
Expand Up @@ -1007,7 +1007,6 @@ def import_scikit_fuzzy() -> str:
assert "matplotlib" in package_set
assert "pyyaml" in package_set
assert "pandas" in package_set
assert "python-dateutil" in package_set
assert "scikit-learn" in package_set
assert "six" in package_set

Expand Down

0 comments on commit 6fb2515

Please sign in to comment.