Skip to content

docs: require data_storage_version=2.2 in map type example#6032

Merged
Xuanwo merged 1 commit intomainfrom
xuanwo/map-docs-v22-example
Feb 27, 2026
Merged

docs: require data_storage_version=2.2 in map type example#6032
Xuanwo merged 1 commit intomainfrom
xuanwo/map-docs-v22-example

Conversation

@Xuanwo
Copy link
Copy Markdown
Collaborator

@Xuanwo Xuanwo commented Feb 27, 2026

Summary

  • update the Map type guide example to explicitly set data_storage_version="2.2"
  • add a short note that Map writes require Lance file format version 2.2 or later

Why

Map writes are only supported with file format 2.2+, and writing with 2.1 fails in tests. The previous example used default settings, which can be copied by users into a failing workflow.


Parts of this PR were drafted with assistance from Codex (with gpt-5.3-codex) and fully reviewed and edited by me. I take full responsibility for all changes.

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Feb 27, 2026
@github-actions
Copy link
Copy Markdown
Contributor

LGTM! This is a clear documentation fix that prevents users from copying an example that would fail with the default settings.

No concerns - docs-only change that improves user experience.

@Xuanwo Xuanwo merged commit b2324c0 into main Feb 27, 2026
8 checks passed
@Xuanwo Xuanwo deleted the xuanwo/map-docs-v22-example branch February 27, 2026 07:42
wjones127 pushed a commit to wjones127/lance that referenced this pull request Mar 4, 2026
…mat#6032)

## Summary
- update the Map type guide example to explicitly set
`data_storage_version="2.2"`
- add a short note that Map writes require Lance file format version 2.2
or later

## Why
Map writes are only supported with file format 2.2+, and writing with
2.1 fails in tests. The previous example used default settings, which
can be copied by users into a failing workflow.

---

**Parts of this PR were drafted with assistance from Codex (with
`gpt-5.3-codex`) and fully reviewed and edited by me. I take full
responsibility for all changes.**
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants