Skip to content

MB-59633: Disable DocValues Chunking & Compression for Geo Fields#2269

Merged
Likith101 merged 23 commits intomasterfrom
geoPerf
Feb 23, 2026
Merged

MB-59633: Disable DocValues Chunking & Compression for Geo Fields#2269
Likith101 merged 23 commits intomasterfrom
geoPerf

Conversation

@Likith101
Copy link
Copy Markdown
Member

@Likith101 Likith101 commented Dec 23, 2025 •
edited
Loading

  • Added docValues, skipSnappy and skipChunking to all geo fields
  • Switched out bytes.Index with bytes.IndexByte
  • Fixed relavent test cases

CascadingRadium
CascadingRadium reviewed Dec 23, 2025
View reviewed changes
Comment thread document/field_geopoint.go
@CascadingRadium CascadingRadium added the geoV2 Geospatial Search V2 label Dec 26, 2025
@CascadingRadium CascadingRadium changed the title MB-59633: Removed snappy encoding and set chunk mode to 1 MB-59633: Skip docvalue chunking and snappy encoding for geopoint and geoshape fields Dec 26, 2025
@CascadingRadium CascadingRadium changed the title MB-59633: Skip docvalue chunking and snappy encoding for geopoint and geoshape fields MB-59633: Disable DocValues Chunking & Compression for Geo Fields Dec 26, 2025
@CascadingRadium CascadingRadium removed the geoV2 Geospatial Search V2 label Dec 26, 2025
@abhinavdangeti abhinavdangeti added this to the v2.6.0 milestone Jan 8, 2026
Base automatically changed from geo to master January 29, 2026 07:15
@CascadingRadium
Copy link
Copy Markdown
Member

CascadingRadium commented Feb 3, 2026

merge master into geoPerf

CascadingRadium
CascadingRadium previously approved these changes Feb 3, 2026
View reviewed changes
CascadingRadium
CascadingRadium approved these changes Feb 19, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@CascadingRadium CascadingRadium left a comment

Choose a reason for hiding this comment

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

need to update deps (bleve_index_api and zapx)

@abhinavdangeti
Copy link
Copy Markdown
Member

abhinavdangeti commented Feb 19, 2026

Looks fine, will add +2 after we merge changes downstream and update the go.mod.

capemox
capemox previously approved these changes Feb 23, 2026
View reviewed changes
@Likith101 Likith101 dismissed stale reviews from capemox and CascadingRadium via 4893011 February 23, 2026 15:40
@coveralls
Copy link
Copy Markdown

coveralls commented Feb 23, 2026

Coverage Status

coverage: 52.668% (-0.02%) from 52.691%
when pulling 434c766 on geoPerf
into f7b13b9 on master.

@Likith101 Likith101 merged commit 465d5fd into master Feb 23, 2026
10 checks passed
@Likith101 Likith101 deleted the geoPerf branch February 23, 2026 16:12
@github-project-automation github-project-automation Bot moved this from Todo to Done in Geo-Spatial Search v2 Feb 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@abhinavdangeti abhinavdangeti abhinavdangeti approved these changes

@CascadingRadium CascadingRadium CascadingRadium approved these changes

@capemox capemox capemox left review comments

@Thejas-bhat Thejas-bhat Awaiting requested review from Thejas-bhat

@maneuvertomars maneuvertomars Awaiting requested review from maneuvertomars

Assignees

No one assigned

Labels

None yet

Projects

No open projects
Geo-Spatial Search v2
Status: Done

Milestone

v2.6.0

Development

Successfully merging this pull request may close these issues.

5 participants

@Likith101 @CascadingRadium @abhinavdangeti @coveralls @capemox