Skip to content

fix: preserve commas in SQL IN lists - #948

Merged
Slach merged 6 commits into
Altinity:masterfrom
dajiaohuang:stew/c1-1536-adhoc-in-list-separators-20261023
Nov 3, 2026
Merged

Slach merged 6 commits into
Altinity:masterfrom
dajiaohuang:stew/c1-1536-adhoc-in-list-separators-20261023

Conversation

@dajiaohuang

Copy link
Copy Markdown
Contributor

Summary

  • Preserve separators when the SQL AST reconstructs multi-value IN (...) lists during ad-hoc filter application.
  • Add regression coverage for string and numeric lists and the filter-injection path.

Fixes #941.

Verification

  • go test ./... -count=1
  • git diff --check

Keep multi-value IN predicates intact when the SQL AST is formatted during ad-hoc filter application. Fixes Altinity#941.
@Slach Slach added this to the 3.6.0 milestone Oct 26, 2026
@Slach
Slach merged commit 07c9842 into Altinity:master Nov 3, 2026
45 checks passed
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.

ad-hoc filters breaking statements including IN()

3 participants