Skip to content

Open and load geo payload indices in single stage#7038

Merged
timvisee merged 3 commits intodevfrom
payload-index-single-stage-load-geo
Aug 13, 2025
Merged

Open and load geo payload indices in single stage#7038
timvisee merged 3 commits intodevfrom
payload-index-single-stage-load-geo

Conversation

@timvisee
Copy link
Member

@timvisee timvisee commented Aug 12, 2025

Same as #6999, but for the geo index.

Tasks

All Submissions:

  • Contributions should target the dev branch. Did you create your branch from dev?
  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

New Feature Submissions:

  1. Does your submission pass tests?
  2. Have you formatted your code locally using cargo +nightly fmt --all command prior to submission?
  3. Have you checked your code using cargo clippy --all --all-features command?

@timvisee timvisee force-pushed the payload-index-single-stage-load-text branch from 58fe3be to 75e8694 Compare August 13, 2025 08:33
Base automatically changed from payload-index-single-stage-load-text to dev August 13, 2025 08:53
@timvisee timvisee force-pushed the payload-index-single-stage-load-geo branch from bf1c348 to 087d119 Compare August 13, 2025 08:54
@timvisee timvisee marked this pull request as ready for review August 13, 2025 08:54
@coderabbitai

This comment was marked as resolved.

Comment on lines 145 to +146
if !index.load()? {
return Ok(false);
return Ok(None);
Copy link
Contributor

Choose a reason for hiding this comment

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

interesting choice of not removing this until #7041 :)

Copy link
Member Author

Choose a reason for hiding this comment

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

🤷‍♂️ wanted to keep these PRs as small as possible, doing things step by step

@timvisee timvisee merged commit c3b5fd0 into dev Aug 13, 2025
16 checks passed
@timvisee timvisee deleted the payload-index-single-stage-load-geo branch August 13, 2025 14:59
timvisee added a commit that referenced this pull request Aug 14, 2025
* Open and load mutable geo index in single stage

* Open and load immutable geo index in single stage

* Open and load mmap geo index in single stage
@coderabbitai coderabbitai bot mentioned this pull request Mar 10, 2026
1 task
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