Skip to content

fix: Master release bug fixes#2485

Merged
BrendanWalsh merged 2 commits into
masterfrom
brwals/master-release-fixes
Feb 4, 2026
Merged

fix: Master release bug fixes#2485
BrendanWalsh merged 2 commits into
masterfrom
brwals/master-release-fixes

Conversation

@BrendanWalsh

Copy link
Copy Markdown
Collaborator

Summary

Backports critical bug fixes from the Spark 4.0 release testing to master.

Changes

1. Azure Maps Spatial API Retired (9/30/2025)

  • CheckPointInPolygon.scala: Throws UnsupportedOperationException on transform()
  • AzureMapsSuite.scala: Removed AzMapsPointInPolygonSuite test class
  • FuzzingTest.scala: Added exemptions for CheckPointInPolygon

2. PEP 440 Python Version Normalization

  • build.scala: pythonizedVersion now uses + local version separator and .toLowerCase() to match pip/wheel filename normalization

3. Conda Environment Activation

  • build.scala: activateCondaEnv now uses conda run on Linux to ensure correct environment

4. Language API Version Update

  • AnalyzeTextSuite.scala, AnalyzeTextLROSuite.scala: Updated from retired 2022-10-01-preview to 2024-11-01

5. R Tests Local Jar Loading

  • RTestGen.scala: Loads all module jars locally instead of relying on Maven resolution of potentially unpublished SNAPSHOTs

Testing

Verified against spark4.0 branch where these fixes have passed CI.

Related

@github-actions

github-actions Bot commented Feb 4, 2026

Copy link
Copy Markdown

Hey @BrendanWalsh 👋!
Thank you so much for contributing to our repository 🙌.
Someone from SynapseML Team will be reviewing this pull request soon.

We use semantic commit messages to streamline the release process.
Before your pull request can be merged, you should make sure your first commit and PR title start with a semantic prefix.
This helps us to create release messages and credit you for your hard work!

Examples of commit messages with semantic prefixes:

  • fix: Fix LightGBM crashes with empty partitions
  • feat: Make HTTP on Spark back-offs configurable
  • docs: Update Spark Serving usage
  • build: Add codecov support
  • perf: improve LightGBM memory usage
  • refactor: make python code generation rely on classes
  • style: Remove nulls from CNTKModel
  • test: Add test coverage for CNTKModel

To test your commit locally, please follow our guild on building from source.
Check out the developer guide for additional guidance on testing your change.

@BrendanWalsh

Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@codecov-commenter

codecov-commenter commented Feb 4, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 84.23%. Comparing base (baf63dc) to head (d2a9dde).

Files with missing lines Patch % Lines
...e/ml/services/geospatial/CheckPointInPolygon.scala 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2485      +/-   ##
==========================================
- Coverage   84.43%   84.23%   -0.21%     
==========================================
  Files         335      335              
  Lines       17779    17781       +2     
  Branches     1603     1592      -11     
==========================================
- Hits        15012    14978      -34     
- Misses       2767     2803      +36     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@BrendanWalsh BrendanWalsh force-pushed the brwals/master-release-fixes branch 2 times, most recently from f0388b2 to 5f85b45 Compare February 4, 2026 17:36
- Handle retired Azure Maps Spatial API (CheckPointInPolygon throws on transform)
- Remove AzMapsPointInPolygonSuite test class and add fuzzing exemptions
- Update Language API version from 2022-10-01-preview to 2024-11-01
- Fix PEP 440 Python wheel version normalization (lowercase + local version)
- Fix conda environment activation on Linux (use conda run)

See: https://azure.microsoft.com/en-us/updates/v2/azure-maps-creator-services-retirement-on-30-september-2025
@BrendanWalsh BrendanWalsh force-pushed the brwals/master-release-fixes branch from 5f85b45 to d2a9dde Compare February 4, 2026 17:44
@BrendanWalsh

Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

Azure Maps Spatial API was retired on 9/30/2025. The notebooks
'Quickstart - Flooding Risk.ipynb' and 'Geospatial Services.ipynb'
use CheckPointInPolygon which calls this retired API.

Exclude these from DatabricksCPUTests to prevent E2E test failures.
@BrendanWalsh BrendanWalsh merged commit 079ec1c into master Feb 4, 2026
6 of 7 checks passed
@BrendanWalsh BrendanWalsh deleted the brwals/master-release-fixes branch February 4, 2026 21:49
BrendanWalsh added a commit that referenced this pull request Feb 5, 2026
* fix: Master release bug fixes

- Handle retired Azure Maps Spatial API (CheckPointInPolygon throws on transform)
- Remove AzMapsPointInPolygonSuite test class and add fuzzing exemptions
- Update Language API version from 2022-10-01-preview to 2024-11-01
- Fix PEP 440 Python wheel version normalization (lowercase + local version)
- Fix conda environment activation on Linux (use conda run)

See: https://azure.microsoft.com/en-us/updates/v2/azure-maps-creator-services-retirement-on-30-september-2025

* test: Exclude geospatial notebooks from E2E tests

Azure Maps Spatial API was retired on 9/30/2025. The notebooks
'Quickstart - Flooding Risk.ipynb' and 'Geospatial Services.ipynb'
use CheckPointInPolygon which calls this retired API.

Exclude these from DatabricksCPUTests to prevent E2E test failures.
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.

2 participants