Skip to content

update for sql-redis 0.5.0#598

Merged
rbs333 merged 1 commit into
mainfrom
feat/sql-redis-0.5.0-regression-tests
May 12, 2026
Merged

update for sql-redis 0.5.0#598
rbs333 merged 1 commit into
mainfrom
feat/sql-redis-0.5.0-regression-tests

Conversation

@rbs333
Copy link
Copy Markdown
Collaborator

@rbs333 rbs333 commented May 6, 2026

Note

Medium Risk
Upgrades the optional sql-redis dependency, which can change SQL-to-RediSearch translation behavior; new regression tests should catch precedence and DISTINCT-count issues but downstream query semantics may still shift.

Overview
Updates the optional dependency sql-redis from 0.4.0 to 0.5.0 (including lockfile updates).

Adds new integration regression coverage for SQLQuery translation/execution, specifically ensuring mixed AND/OR WHERE clauses preserve parenthesized boolean precedence and that COUNT(DISTINCT ...) maps to RediSearch COUNT_DISTINCT (with validation for unsupported DISTINCT forms).

Reviewed by Cursor Bugbot for commit fd2061b. Bugbot is set up for automated code reviews on this repo. Configure here.

@jit-ci
Copy link
Copy Markdown

jit-ci Bot commented May 6, 2026

🛡️ Jit Security Scan Results

CRITICAL HIGH MEDIUM

✅ No security findings were detected in this PR


Security scan by Jit

@rbs333 rbs333 added auto:release Create a release when this PR is merged auto:patch Increment the patch version when merged labels May 12, 2026
@rbs333 rbs333 merged commit 78d2533 into main May 12, 2026
61 checks passed
@rbs333 rbs333 deleted the feat/sql-redis-0.5.0-regression-tests branch May 12, 2026 15:38
@applied-ai-release-bot
Copy link
Copy Markdown

🚀 PR was released in v0.18.2 🚀

@applied-ai-release-bot applied-ai-release-bot Bot added the released This issue/pull request has been released. label May 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto:patch Increment the patch version when merged auto:release Create a release when this PR is merged released This issue/pull request has been released.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants