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

Check that the Java / Scala package is installed when needed #250

Merged
merged 1 commit into from
Aug 16, 2024

Conversation

EnricoMi
Copy link
Contributor

@EnricoMi EnricoMi commented Aug 15, 2024

Provides a meaningful error message when user accesses a spark extension function in Python that requires the Java / Scala package:

RuntimeError: Java / Scala package not found! You need to add the Maven spark-extension package to your PySpark environment: https:/G-Research/spark-extension#python

Before, the error was:

TypeError: 'JavaPackage' object is not callable

Improves #242
Supersedes #244

Copy link

github-actions bot commented Aug 15, 2024

Test Results

   624 files   -  13     624 suites   - 13   1h 46m 42s ⏱️ + 1m 49s
   511 tests +  1     511 ✅ +  1    0 💤 ±0  0 ❌ ±0 
19 580 runs   - 413  19 016 ✅  - 419  564 💤 +6  0 ❌ ±0 

Results for commit 0eaaa7d. ± Comparison against base commit 3a046ca.

♻️ This comment has been updated with latest results.

Base automatically changed from support-spark-connect to master August 16, 2024 18:16
@EnricoMi EnricoMi disabled auto-merge August 16, 2024 18:22
@EnricoMi EnricoMi added this pull request to the merge queue Aug 16, 2024
Merged via the queue into master with commit e9e508d Aug 16, 2024
90 of 91 checks passed
@EnricoMi EnricoMi deleted the check-java-pkg-installed-2 branch August 17, 2024 08:49
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.

1 participant