Skip to content

MB-72376: Introducing New Geo-Shape Indexes - #31

Merged
Likith101 merged 10 commits into
masterfrom
geo
Aug 3, 2026
Merged

MB-72376: Introducing New Geo-Shape Indexes#31
Likith101 merged 10 commits into
masterfrom
geo

Conversation

@Likith101

@Likith101 Likith101 commented Jun 26, 2026

Copy link
Copy Markdown
Member
  • All Geo-Shapes will also implement IndexCells(), QueryCells() and BoundingBox() for v2 support
  • Introduced new helper methods within geojson_v2.go
  • Moved all shape specific code and test cases to their respective files for modularity
  • Added additional test cases for v2

@Samsonnyyeet Samsonnyyeet left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Only queries I had were related to code that was already part of the codebase and was only a part of the review because of file restructuring. No comments on the new code.

@Likith101
Likith101 merged commit c3865c4 into master Aug 3, 2026
12 checks passed
@Likith101
Likith101 deleted the geo branch August 3, 2026 10:14
@github-project-automation github-project-automation Bot moved this from Todo to Done in Geo-Shape V2 Aug 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants