Skip to content

DCV-3638: Detect TOP and FETCH in has_limit#18

Open
jesusmgg wants to merge 1 commit intomainfrom
DCV-3638
Open

DCV-3638: Detect TOP and FETCH in has_limit#18
jesusmgg wants to merge 1 commit intomainfrom
DCV-3638

Conversation

@jesusmgg
Copy link

@jesusmgg jesusmgg commented Feb 12, 2026

Summary

  • Expand has_limit regex to detect SELECT TOP N and FETCH FIRST/NEXT N ROWS ONLY
    in addition to LIMIT N, avoiding unnecessary query wrapping

Extension-side fix: datacoves/datacoves-power-user#68

Test plan

  • LIMIT queries pass through as-is (unchanged behavior)
  • TOP queries now pass through instead of being wrapped
  • FETCH queries now pass through instead of being wrapped
  • No-limit queries still get wrapped with adapter-specific limit

After merge

git tag v1.1.6 && git push origin v1.1.6

@jesusmgg jesusmgg self-assigned this Feb 12, 2026
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

Comments