Skip to content

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

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

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

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants