Skip to content

Support limit n,m#4792

Merged
bohutang merged 1 commit into
databendlabs:mainfrom
TCeason:ISSUE-4709/support_limit_nm
Apr 11, 2022
Merged

Support limit n,m#4792
bohutang merged 1 commit into
databendlabs:mainfrom
TCeason:ISSUE-4709/support_limit_nm

Conversation

@TCeason

@TCeason TCeason commented Apr 11, 2022

Copy link
Copy Markdown
Collaborator

I hereby agree to the terms of the CLA available at: https://databend.rs/dev/policies/cla/

Summary

Support select query :

select * from foo limit n, m;

Changelog

  • Improvement

Related Issues

Fixes #4709

@vercel

vercel Bot commented Apr 11, 2022

Copy link
Copy Markdown

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/databend/databend/5MDN8ewx4Kr4grHjLKAMQqnJ2UWt
✅ Preview: Ignored

[Deployment for bda648d canceled]

@mergify

mergify Bot commented Apr 11, 2022

Copy link
Copy Markdown
Contributor

Thanks for the contribution!
I have applied any labels matching special text in your PR Changelog.

Please review the labels and make any necessary changes.

@TCeason
TCeason force-pushed the ISSUE-4709/support_limit_nm branch from 5db67ed to 06e197e Compare April 11, 2022 07:04
@sundy-li

sundy-li commented Apr 11, 2022

Copy link
Copy Markdown
Member

Need stateless tests

@TCeason
TCeason force-pushed the ISSUE-4709/support_limit_nm branch from 06e197e to 80a3999 Compare April 11, 2022 07:21
@TCeason

TCeason commented Apr 11, 2022

Copy link
Copy Markdown
Collaborator Author

Need stateless tests

Added.

@TCeason
TCeason force-pushed the ISSUE-4709/support_limit_nm branch from 80a3999 to bda648d Compare April 11, 2022 07:55
@bohutang
bohutang requested a review from sundy-li April 11, 2022 08:33
@bohutang
bohutang merged commit cc17e98 into databendlabs:main Apr 11, 2022
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.

Feature: Support limit n, m

4 participants