Skip to content

redis search: fix FAST and rrf samples - #987

Draft
ytkimirti wants to merge 2 commits into
mainfrom
DX-3247-search-types
Draft

ytkimirti wants to merge 2 commits into
mainfrom
DX-3247-search-types

Conversation

@ytkimirti

@ytkimirti ytkimirti commented Nov 3, 2026 •

Copy link
Copy Markdown
Collaborator

Preview: https://upstash-docs-git-dx-3247-search-types-4b82b0.docs7.io/redis/search/troubleshooting

Fixes search samples that fail on the server or under the tighter redis-js search types: Python schemas that sort by a plain F64 (not FAST), the troubleshooting note that says s.date() and s.boolean() are FAST by default, and the rrf.mdx paging filter, which needs as const to keep its literal field names.

SDK: upstash/redis-js#1467, upstash/redis-py#79. Merge this first.

Evidence: the Python schema from getting-started.mdx against a live search DB, sorting by price:

FAST:  [QueryResult(key='dxchk:2', ... 'price': 1}), QueryResult(key='dxchk:1', ... 'price': 3})]
plain: Schema error: 'Field `price` is not a fast field.'

The rrf.mdx paging snippet type-checks against the redis-js branch with as const (tsc exit 0) and fails without it (TS2322).

@mintlify

mintlify Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated
upstash 🟢 Ready View Preview Nov 3, 2026, 12:12 PM

💡 Tip: Enable Automations to automatically generate PRs for you.

@linear-code

linear-code Bot commented Nov 3, 2026

Copy link
Copy Markdown

DX-3247

@context7

context7 Bot commented Nov 3, 2026 •

Copy link
Copy Markdown
Contributor

Docs7 for upstash.com

Result Status Action
Deployment ✅ Ready Open preview
Content review ✅ Passed. No problems found. —

Commit de3148c · Updated 2026-11-03 12:14 UTC · View build details

This branch was successfully deployed

2 active deployments
Docs7 Preview: DX-3247-search-types — de3148c3 Deployed Nov 3, 2026 by context7[bot]
staging — de3148c3 Deployed Nov 3, 2026 by mintlify[bot]
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