Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Backport 2.x] Pagination Phase 2: Support WHERE clause, column list in SELECT clause and for functions and expressions in the query. #1741

Merged
merged 1 commit into from
Jun 15, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport da386e5 from #1500

…lause and for functions and expressions in the query. (#1500)

* Add support for `WHERE` clause, column list in `SELECT` clause and for functions and expressions in the query.

Signed-off-by: Yury-Fridlyand <[email protected]>

* Fix merge issue and address PR feedback by updating comments.

Signed-off-by: Yury-Fridlyand <[email protected]>

* More comments.

Signed-off-by: Yury-Fridlyand <[email protected]>

* Add extra check for unset `initialSearchRequest`.

Signed-off-by: Yury-Fridlyand <[email protected]>

---------

Signed-off-by: Yury-Fridlyand <[email protected]>
(cherry picked from commit da386e5)
@codecov
Copy link

codecov bot commented Jun 14, 2023

Codecov Report

Merging #1741 (061b940) into 2.x (c23a788) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##                2.x    #1741   +/-   ##
=========================================
  Coverage     97.30%   97.31%           
- Complexity     4328     4371   +43     
=========================================
  Files           385      385           
  Lines         10809    10845   +36     
  Branches        762      763    +1     
=========================================
+ Hits          10518    10554   +36     
  Misses          284      284           
  Partials          7        7           
Flag Coverage Δ
sql-engine 97.31% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...sql/opensearch/request/OpenSearchQueryRequest.java 100.00% <ø> (ø)
...arch/sql/opensearch/request/OpenSearchRequest.java 100.00% <ø> (ø)
...ch/sql/executor/pagination/CanPaginateVisitor.java 100.00% <100.00%> (ø)
...ql/opensearch/request/OpenSearchScrollRequest.java 100.00% <100.00%> (ø)

@Yury-Fridlyand Yury-Fridlyand merged commit d2a82ae into 2.x Jun 15, 2023
@github-actions github-actions bot deleted the backport/backport-1500-to-2.x branch June 15, 2023 17:49
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.

4 participants