Skip to content

docs: expand README limitations and fix stale docs#287

Merged
tjgreen42 merged 4 commits intomainfrom
docs/readme-limitations-and-fixes
Mar 19, 2026
Merged

docs: expand README limitations and fix stale docs#287
tjgreen42 merged 4 commits intomainfrom
docs/readme-limitations-and-fixes

Conversation

@tjgreen42
Copy link
Copy Markdown
Collaborator

@tjgreen42 tjgreen42 commented Mar 13, 2026

Summary

  • Add missing limitation sections to README for GA: no phrase queries, no expression indexing, no built-in faceted search, insert/update performance, no background compaction
  • Each limitation includes SQL examples showing workarounds
  • Fix incorrect "LZ4 compression" in CHANGELOG (actual: delta encoding + bitpacking)
  • Update stale parallel build description in CLAUDE.md (old page pool → BufFile architecture)

Testing

Documentation-only changes. Verified SQL examples match existing patterns in the README.

Add missing limitation sections to README: no phrase queries (with
subquery workaround), no expression indexing (generated column
workaround), no built-in faceted search (WHERE/GROUP BY examples),
insert/update performance status, and no background compaction.

Also fixes:
- CHANGELOG: incorrect "LZ4 compression" → delta encoding + bitpacking
- CLAUDE.md: stale parallel build description (old page pool → BufFile
  architecture) and remove obsolete pool-exhaustion limitation
CREATE INDEX always produces a single segment, so bm25_force_merge()
is not needed after index builds. It is useful after sustained
incremental inserts where repeated memtable spills create multiple
segments.
@tjgreen42 tjgreen42 marked this pull request as ready for review March 18, 2026 01:26
- Fix incorrect "CREATE INDEX always produces a single segment" claim
  (serial builds can produce multiple segments)
- Add COALESCE to generated column example to handle NULLs
- Add ORDER BY + score alias to phrase query outer query
- Fix faceted search comment to note B-tree index is assumed
- Update stale source comments: remove page pool reference in
  segment.c, fix force_merge docstrings in build.c and merge.h
- Simplify CLAUDE.md parallel build step 5
@tjgreen42 tjgreen42 merged commit 4b3d725 into main Mar 19, 2026
14 checks passed
@tjgreen42 tjgreen42 deleted the docs/readme-limitations-and-fixes branch March 19, 2026 00:08
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.

1 participant