Skip to content

update search param docs#14

Open
andreyorst wants to merge 3 commits into
mainfrom
smart-search-params
Open

update search param docs#14
andreyorst wants to merge 3 commits into
mainfrom
smart-search-params

Conversation

@andreyorst
Copy link
Copy Markdown
Contributor

changes related to smart search params in Aidbox

@andreyorst andreyorst requested a review from spicyfalafel May 14, 2026 09:11
Comment thread docs/deployment-and-maintenance/indexes/get-suggested-indexes.md Outdated
Comment thread docs/deployment-and-maintenance/indexes/get-suggested-indexes.md Outdated
Comment thread docs/deployment-and-maintenance/indexes/get-suggested-indexes.md Outdated
Comment thread docs/deployment-and-maintenance/indexes/get-suggested-indexes.md Outdated
Comment thread docs/deployment-and-maintenance/indexes/README.md Outdated
Comment thread docs/deployment-and-maintenance/indexes/search-parameter-usage-stats.md Outdated
Comment thread docs/deployment-and-maintenance/indexes/search-parameter-usage-stats.md Outdated
Comment thread docs/deployment-and-maintenance/indexes/search-parameter-usage-stats.md Outdated
Comment thread docs/deployment-and-maintenance/indexes/search-parameter-usage-stats.md Outdated
Comment thread docs/deployment-and-maintenance/indexes/search-parameter-usage-stats.md Outdated
@andreyorst andreyorst requested a review from spicyfalafel May 15, 2026 12:01
@andreyorst andreyorst force-pushed the smart-search-params branch from 6e9fcdf to 640aa0a Compare May 15, 2026 12:22
Comment thread docs/deployment-and-maintenance/indexes/get-suggested-indexes.md Outdated
Comment thread docs/deployment-and-maintenance/indexes/README.md Outdated
Comment thread docs/deployment-and-maintenance/indexes/README.md Outdated
Comment thread docs/deployment-and-maintenance/indexes/README.md Outdated
Comment thread docs/deployment-and-maintenance/indexes/README.md
Comment thread docs/deployment-and-maintenance/indexes/README.md Outdated
Comment thread docs/deployment-and-maintenance/indexes/README.md Outdated
Comment thread docs/deployment-and-maintenance/indexes/search-parameter-usage-stats.md Outdated
Comment thread docs/deployment-and-maintenance/indexes/search-parameter-usage-stats.md Outdated
Comment thread docs/deployment-and-maintenance/indexes/search-parameter-usage-stats.md Outdated
Comment thread docs/deployment-and-maintenance/indexes/search-parameter-usage-stats.md Outdated
Comment thread docs/deployment-and-maintenance/indexes/search-parameter-usage-stats.md Outdated
Comment thread docs/deployment-and-maintenance/indexes/search-parameter-usage-stats.md Outdated
Comment thread docs/deployment-and-maintenance/indexes/get-suggested-indexes.md Outdated
Comment thread docs/deployment-and-maintenance/indexes/search-parameter-usage-stats.md Outdated
@andreyorst andreyorst force-pushed the smart-search-params branch 2 times, most recently from f3047a3 to f28b8fa Compare May 15, 2026 14:32
@andreyorst andreyorst force-pushed the smart-search-params branch from f28b8fa to c12315a Compare May 15, 2026 14:50
@andreyorst andreyorst requested a review from spicyfalafel May 15, 2026 14:50
* **Modifiers.** `:contains` and `:exact` on the same `name` parameter need different functional indexes; `:in` / `:not-in` / `:above` / `:below` on token parameters expand into ValueSet lookups; `:identifier` / `:of-type` pull from different jsonb paths.
* **Path expressions.** Aidbox stores resources as jsonb, so the suggester emits *functional* indexes over `knife_extract_text(...)` or `jsonb_path_query(...)` — one per SP path — rather than indexes on plain columns.
* **Joins.** Chained queries (`Observation?subject:Patient.name=John`) and reverse-chain `_has` queries translate into SQL joins or subselects; both sides need their own indexes.
* **Full-resource fallback.** Token and reference parameters without a dedicated path fall back to a GIN over the whole jsonb (`<rt>_resource_jsonb`). It rescues queries that no functional index covers, but it's larger on disk.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

remove (<rt>_resource_jsonb)

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.

2 participants