Skip to content

Support OFFSET LIMIT as well as LIMIT OFFSET#413

Merged
alamb merged 9 commits intoapache:mainfrom
alamb:alamb/offset
Feb 5, 2022
Merged

Support OFFSET LIMIT as well as LIMIT OFFSET#413
alamb merged 9 commits intoapache:mainfrom
alamb:alamb/offset

Conversation

@alamb
Copy link
Contributor

@alamb alamb commented Feb 5, 2022

Based on #385 frmo @yuval-illumex merged to master so CI tests can pass and a test to ensure we can't duplicate LIMIT and OFFSET

@coveralls
Copy link

Pull Request Test Coverage Report for Build 1799427648

  • 27 of 27 (100.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.03%) to 90.32%

Totals Coverage Status
Change from base Build 1799400715: 0.03%
Covered Lines: 6746
Relevant Lines: 7469

💛 - Coveralls

@alamb alamb merged commit 33d4f27 into apache:main Feb 5, 2022
@alamb alamb deleted the alamb/offset branch February 5, 2022 12:46
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.

3 participants