Skip to content

Remove flex locations from geocoder#7339

Merged
leonardehrenfried merged 2 commits intoopentripplanner:dev-2.xfrom
ibi-group:remove-flex-geocoder
Feb 27, 2026
Merged

Remove flex locations from geocoder#7339
leonardehrenfried merged 2 commits intoopentripplanner:dev-2.xfrom
ibi-group:remove-flex-geocoder

Conversation

@leonardehrenfried
Copy link
Member

Summary

This PR remove the flex locations from the Lucene index used in the geocoder API.

It also removes some duplicate indexing of entities that are no longer queried.

Changelog

Skip.

Bumping the serialization version id

No.

@leonardehrenfried leonardehrenfried self-assigned this Feb 24, 2026
@leonardehrenfried leonardehrenfried requested a review from a team as a code owner February 24, 2026 17:04
@leonardehrenfried leonardehrenfried added +Sandbox This will be implemented as a Sandbox feature +Skip Changelog This is not a relevant change for a product owner since last release. labels Feb 24, 2026
@codecov
Copy link

codecov bot commented Feb 24, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.80%. Comparing base (aaca490) to head (fb4e79b).
⚠️ Report is 18 commits behind head on dev-2.x.

Additional details and impacted files
@@              Coverage Diff              @@
##             dev-2.x    #7339      +/-   ##
=============================================
- Coverage      70.80%   70.80%   -0.01%     
- Complexity     20710    20713       +3     
=============================================
  Files           2325     2325              
  Lines          86270    86253      -17     
  Branches        8532     8532              
=============================================
- Hits           61084    61070      -14     
  Misses         22222    22222              
+ Partials        2964     2961       -3     

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

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Contributor

@binh-dam-ibigroup binh-dam-ibigroup left a comment

Choose a reason for hiding this comment

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

Works as intended.

@leonardehrenfried leonardehrenfried merged commit ac0610d into opentripplanner:dev-2.x Feb 27, 2026
8 checks passed
@leonardehrenfried leonardehrenfried deleted the remove-flex-geocoder branch February 27, 2026 06:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

+Sandbox This will be implemented as a Sandbox feature +Skip Changelog This is not a relevant change for a product owner since last release.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants