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] add concurrent limit on datasource and sessions #2394

Merged
merged 1 commit into from
Oct 30, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport d3ce049 from #2390.

* add concurrent limit on datasource and sessions

Signed-off-by: Peng Huo <[email protected]>

* fix ut coverage

Signed-off-by: Peng Huo <[email protected]>

---------

Signed-off-by: Peng Huo <[email protected]>
(cherry picked from commit d3ce049)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@codecov
Copy link

codecov bot commented Oct 30, 2023

Codecov Report

Merging #2394 (5661e00) into 2.x (56bd9dc) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##                2.x    #2394      +/-   ##
============================================
- Coverage     95.61%   95.60%   -0.01%     
- Complexity     4955     4959       +4     
============================================
  Files           469      472       +3     
  Lines         13835    13842       +7     
  Branches        918      924       +6     
============================================
+ Hits          13228    13234       +6     
  Misses          587      587              
- Partials         20       21       +1     
Flag Coverage Δ
sql-engine 95.60% <100.00%> (-0.01%) ⬇️

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

Files Coverage Δ
...ces/transport/TransportCreateDataSourceAction.java 100.00% <100.00%> (ø)
...rch/sql/opensearch/setting/OpenSearchSettings.java 100.00% <100.00%> (ø)
...search/sql/spark/dispatcher/AsyncQueryHandler.java 100.00% <ø> (ø)
...search/sql/spark/dispatcher/BatchQueryHandler.java 100.00% <100.00%> (ø)
.../sql/spark/dispatcher/InteractiveQueryHandler.java 100.00% <100.00%> (ø)
...rch/sql/spark/dispatcher/SparkQueryDispatcher.java 100.00% <100.00%> (ø)
...ch/sql/spark/dispatcher/StreamingQueryHandler.java 100.00% <100.00%> (ø)
...ch/sql/spark/execution/session/SessionManager.java 100.00% <ø> (ø)
...rch/sql/spark/execution/statestore/StateStore.java 81.75% <100.00%> (-0.27%) ⬇️
...easemanager/ConcurrencyLimitExceededException.java 100.00% <100.00%> (ø)
... and 1 more

... and 1 file with indirect coverage changes

@penghuo
Copy link
Collaborator

penghuo commented Oct 30, 2023

Flaky test: string format issue. will fix in #2386

@penghuo penghuo merged commit 1fd492e into 2.x Oct 30, 2023
31 of 34 checks passed
@github-actions github-actions bot deleted the backport/backport-2390-to-2.x branch October 30, 2023 20:07
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