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

Removed "packaging" package dependency #3469

Merged
merged 1 commit into from
Dec 23, 2024
Merged

Conversation

FastLee
Copy link
Contributor

@FastLee FastLee commented Dec 23, 2024

Removed dependency on "packaging" package to address release issue.

@FastLee FastLee requested a review from a team as a code owner December 23, 2024 21:43
@FastLee FastLee enabled auto-merge December 23, 2024 21:43
Copy link
Contributor

@pritishpai pritishpai left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link

✅ 55/55 passed, 4 skipped, 2h12m57s total

Running from acceptance #7835

@FastLee FastLee added this pull request to the merge queue Dec 23, 2024
Merged via the queue into main with commit 078b418 Dec 23, 2024
7 checks passed
@FastLee FastLee deleted the fix/packaging_dependency branch December 23, 2024 22:08
@KennyShaevelA8
Copy link

@FastLee @pritishpai is there a reason that running databricks labs install ucx seems to not pull down this latest commit of the file?

I am getting error ModuleNotFoundError: No module named 'packaging' and inspecting the file that gets downloaded/unpacked from the zipball shows the version from the previous commit.

@pritishpai
Copy link
Contributor

@KennyShaevelA8 databricks labs install ucx will pull the latest released version. For this change to be pulled, you will have to install the unreleased version with databricks labs install ucx@main

gueniai added a commit that referenced this pull request Dec 30, 2024
* Removed `packaging` package dependency ([#3469](#3469)). In this release, we have removed the dependency on the `packaging` package in the open-source library to address a release issue. The import statements for "packaging.version.Version" and "packaging.version.InvalidVersion" have been removed. The function _external_hms in the federation.py file has been updated to retrieve the Hive Metastore version using the "spark.sql.hive.metastore.version" configuration key and validate it using a regular expression pattern. If the version is not valid, the function logs an informational message and returns None. This change modifies the Hive Metastore version validation logic and improves the overall reliability and maintainability of the library.
@gueniai gueniai mentioned this pull request Dec 30, 2024
gueniai added a commit that referenced this pull request Dec 30, 2024
* Removed `packaging` package dependency
([#3469](#3469)). In this
release, we have removed the dependency on the `packaging` package in
the open-source library to address a release issue. The import
statements for "packaging.version.Version" and
"packaging.version.InvalidVersion" have been removed. The function
_external_hms in the federation.py file has been updated to retrieve the
Hive Metastore version using the "spark.sql.hive.metastore.version"
configuration key and validate it using a regular expression pattern. If
the version is not valid, the function logs an informational message and
returns None. This change modifies the Hive Metastore version validation
logic and improves the overall reliability and maintainability of the
library.
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.

3 participants